Skip to content
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

Add labels #329

Merged
merged 4 commits into from Aug 17, 2021
Merged

Add labels #329

merged 4 commits into from Aug 17, 2021

Conversation

OR13
Copy link
Contributor

@OR13 OR13 commented Aug 12, 2021

Copy link
Member

@msporny msporny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 to this, thanks @OR13. I believe there is some ReSpec magic we could employ to translate a class into HTML, such that all we'd need to do is something like class=label-deprecated on an HTML element and it would do the magic to inject the span plus class plus appropriate text. That is more engineering than we need right now, but might be useful to the editors over time (to avoid excessive boilerplate copy-pasting)... Just noting it, we don't need to put in that effort right now.

@OR13 OR13 merged commit b2f2249 into main Aug 17, 2021
@iherman
Copy link
Member

iherman commented Aug 17, 2021

The issue was discussed in a meeting on 2021-08-17

  • no resolutions were taken
View the transcript

6.1. Add labels (pr did-spec-registries#329)

See github pull request did-spec-registries#329.

Orie Steele: I was absent from the call when we discussed labels, but my understanding is we want to support labels in registry entries.

Brent Zundel: Any opinions on this?
… No opposition to this plan.

Orie Steele: I will open a PR to apply these labels to all the DID methods.

Joe Andrieu: great. Thanks, Orie

@msporny msporny deleted the feat/add-labels-example branch November 14, 2021 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants