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.

Sign up for GitHub

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

Jump to bottom

"core object types and properties" is confusing #634

Open
evanp opened this issue Jan 24, 2025 · 2 comments
Open

"core object types and properties" is confusing #634

evanp opened this issue Jan 24, 2025 · 2 comments
Labels
Needs Group Input/Decision Needs primer page Need to add a page at https://www.w3.org/wiki/Activity_Streams/Primer on this topic

Comments

@evanp
Copy link
Collaborator

evanp commented Jan 24, 2025

In AS2 Core Section 4, the first sentence says, "The Activity Vocabulary normatively defines the core object types and properties for Activity Streams 2.0."

However, in Vocab, there is a set of Core Types (capitalized) and no Core Properties.

I think the use of "core" here might be covering the entire Activity Vocabulary (all types and properties), or it might only be talking about the 8 Core Types.

This should be clarified.
@evanp evanp added Needs erratum We need to add an erratum Needs primer page Need to add a page at https://www.w3.org/wiki/Activity_Streams/Primer on this topic labels Jan 24, 2025
@evanp
Copy link
Collaborator Author

evanp commented Jan 24, 2025

Originated in discussions of #633

evanp added a commit that referenced this issue Feb 7, 2025
@evanp
fix: erratum for "core object types" language in section 4 "Model"
98fa6eb 
Per #634 , the use of the term "core" in the opening sentence of this section is misleading, since it covers both types and properties. The distinction between core types and extended types is made later in the section, so this erratum removes the word "core" from the first sentence.
@evanp evanp mentioned this issue Feb 7, 2025
Open
@evanp
Copy link
Collaborator Author

evanp commented Feb 7, 2025

I've made a PR to add this erratum. We need group consensus to apply this erratum, so I will label "needs group input" and drop "needs erratum".

@evanp evanp added Needs Group Input/Decision and removed Needs erratum We need to add an erratum labels Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Needs Group Input/Decision Needs primer page Need to add a page at https://www.w3.org/wiki/Activity_Streams/Primer on this topic
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@evanp