Add note about mandatory/optional #22
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
TallTed
reviewed
Jun 17, 2025
| @@ -732,6 +732,9 @@ <h1>Overview of List Operator</h1> | |||
| <p>The List Operator provides a framework for managing a trusted list, which includes essential information about the entity responsible for the list's maintenance and its associated attributes.</p> | |||
|
|
|||
| <h2>Attributes of List Operator</h2> | |||
| <p> | |||
| NOTE. It still needs to be decided which attributes should be mandatory and which should be optional. This needs further discussion. | |||
There was a problem hiding this comment.
Wording change, and perhaps this should be an issue, with link in the document?
Suggested change
| NOTE. It still needs to be decided which attributes should be mandatory and which should be optional. This needs further discussion. | |
| NOTE. It is yet to be decided which attributes should be mandatory and which | |
| should be optional. This needs further discussion. |
There was a problem hiding this comment.
Given that this draft has not yet been adopted by the WG, we cannot say it needs further discussion by the WG. The current wording implies that whoever works on the document next, will need to discuss this. So it implies the WG if the WG adopts it.
There was a problem hiding this comment.
revised wording accepted but cannot be committed as the PR has already been merged. So can you raise a new PR please.
| @@ -768,6 +772,9 @@ <h1>Overview of Trust Service Provider</h1> | |||
| <li>A set of URIs from which information about the TSP can be obtained</li> | |||
| </ul> | |||
| <h2>Attributes of Trust Service Provider</h2> | |||
| <p> | |||
| NOTE. It still needs to be decided which attributes should be mandatory and which should be optional. This needs further discussion. | |||
There was a problem hiding this comment.
Wording change, and perhaps this should be an issue, with link in the document?
Suggested change
| NOTE. It still needs to be decided which attributes should be mandatory and which should be optional. This needs further discussion. | |
| NOTE. It is yet to be decided which attributes should be mandatory and which | |
| should be optional. This needs further discussion. |
hendersonweb
added a commit
to hendersonweb/verifiable-issuers-verifiers
that referenced
this pull request
Jul 7, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Preview | Diff