-
Notifications
You must be signed in to change notification settings - Fork 59
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
Is there a minor mistake in the Remove example? #534
Comments
It seems to be that Remove object from location. Add object from location to location. |
I came here to open exactly this issue. The description text is correct, but the Remove example was probably wrongly copy pasted from the Add example. |
So, it seems like an error in the example code. The correct resolution is to add an erratum that says "target" should be "origin" in this example. Unfortunately, ActivityPub defines the Remove activity to use "target". So, we will probably need to recommend that implementers support "origin" by default, and "target" as a fallback, regardless of the unfortunate illogic of it. |
Should this (
activitystreams/test/vocabulary-ex28-jsonld.json
Line 11 in 54a6723
See:
activitystreams/vocabulary/index.html
Line 1510 in 54a6723
The text was updated successfully, but these errors were encountered: