-
Notifications
You must be signed in to change notification settings - Fork 99
Clarify restrictions on identifiers used in DID documents. #886
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
I wonder why the preview did not automatically run. @msporny any idea? |
Yes, the PR Preview App isn't installed for this repo. Perhaps it was desynchronized when we renamed the repo? For some reason, I don't have admin privileges on this repo, so I can't install it. You might be able to install it by going here: https://github.com/apps/pr-preview ... and installing it on this repo specifically. I think you need to have admin privileges on the repo. PS: Looks like Echidna auto-publishing had been disabled for our FPWD and then never turned back on. I have re-enabled it. |
Actually... it works again on another repo! I have no idea why it has not worked for this one. It may have been a temporary glitch. @deniak sorry for the noise! |
85626fc
to
7f0c539
Compare
Hmm, the only thing I changed was re-enabling Echinda publishing (which is now failing for different reasons):
The other thing that's confusing is that the "install" process says it's not enabled for this repo, but perhaps that's because I don't have management permissions on this repo? |
It is an intellectual illness to try to understand everything... :-) |
The issue is not related to Echidna but to respec. There is a problem with the cross references but I don't know respec enough to find what is the problem.
|
@deniak wrote:
Yes, the problem is that the CID spec isn't being indexed in webref, probably because it's not listed in browser-specs. I've raised these two PRs in an attempt to fix the issue:
@iherman wrote:
I hope to one day be cured of this illness... at least, as it relates to tracking down ReSpec build failures... :P |
Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
Editorial, multiple reviews, changes requested and made, no objections, merging. |
This was discussed during the did meeting on 03 April 2025. View the transcriptw3c/did#886<ottomorac> Clarify restrictions on identifiers used in DID documents. #886 ottomorac: Same sort of situation -- Ted had some changes that have been merged. manu: This is asking when parameters should be dropped. <denkeni> +1 to make it clear in did spec ottomorac: This one makes sense too. TallTed: I'll take a quick look at it, but expect it'll be fine. manu: There's a triple-bracket syntax for references. ottomorac: I think we're nearly ready on it, maybe get final approval from Ted and then wrap it up. |
This PR is an attempt to address issue #337 by clarifying specific existing (and presumed) restrictions on DID Document identifiers (such as parameters and fragment identifiers).
Preview | Diff