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

Mention explicitly that Link#name can also be a nameMap #486

Open
1 of 5 tasks
aschrijver opened this issue Jul 4, 2018 · 6 comments
Open
1 of 5 tasks

Mention explicitly that Link#name can also be a nameMap #486

aschrijver opened this issue Jul 4, 2018 · 6 comments

Comments

@aschrijver
Copy link

aschrijver commented Jul 4, 2018

Please Indicate One:

  • Editorial
  • Question
  • Feedback
  • Blocking Issue
  • Non-Blocking Issue

Please Describe the Issue in https://www.w3.org/TR/activitystreams-core:

  • Link#name:

    • Whereas for Object there is explicit mention in ActivityStreams-core that you can use nameMap, in the section describing Link (par. 4.2), there is only mention of name, while the definition of name in Vocabulary states that nameMap property is valid for Links as well. Should mention that in core spec too, for clarity.

As an editorial issue, it would be better to explicitly mention this in the document, IMHO.

@gobengo
Copy link
Contributor

gobengo commented Sep 12, 2018

Will you make a concrete PR that adds the clarity you think would be better?

@aschrijver
Copy link
Author

Yes, I could, however:

Pull requests will be accepted only if the
submitter is a current member of the W3C Social WG but
issues can be opened by anyone.

And I am not a member.

@gobengo
Copy link
Contributor

gobengo commented Sep 14, 2018 via email

@aschrijver
Copy link
Author

Thanks. I made the changes (also reordered properties on Object definition to be alphabetically.

@evanp
Copy link
Collaborator

evanp commented Nov 1, 2023

Based on this suggestion, in issue triage, we reviewed all the properties with domain Link in the vocabulary document. nameMap, preview, and attributedTo were all left out. To resolve this, I created an erratum for section 4.2 of core that gives the full list of properties for a Link, except href which is specified earlier in the paragraph.

The PR is here: #553

I'll try to get this included. We have a lot of ERRATA backed up!

@evanp
Copy link
Collaborator

evanp commented Sep 13, 2024

This should also be added to the SocialCG agenda for next month.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants