activitystreams icon indicating copy to clipboard operation
activitystreams copied to clipboard

Mention explicitly that Link#name can also be a nameMap

Open aschrijver opened this issue 7 years ago • 6 comments

Please Indicate One:

  • [x] 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.

aschrijver avatar Jul 04 '18 12:07 aschrijver

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

gobengo avatar Sep 12 '18 16:09 gobengo

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.

aschrijver avatar Sep 13 '18 09:09 aschrijver

The WG has wound down since a few months ago so I don't think you should pay attention to that.

On Thu, Sep 13, 2018, 10:17 AM Arnold Schrijver [email protected] wrote:

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.

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/w3c/activitystreams/issues/486#issuecomment-420940443, or mute the thread https://github.com/notifications/unsubscribe-auth/AAKfBtYeZ5Dsl7igUPr24KuMh94TFsekks5uaiKRgaJpZM4VCenP .

gobengo avatar Sep 14 '18 01:09 gobengo

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

aschrijver avatar Sep 14 '18 09:09 aschrijver

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: https://github.com/w3c/activitystreams/pull/553

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

evanp avatar Nov 01 '23 16:11 evanp