Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
setup announcement actor, add endpoint for actor #42
setup announcement actor, add endpoint for actor #42
Changes from 32 commits
5320e9f
5f04cc2
3d25d1a
58db44c
5687b42
0051dea
1deddd8
7ed3030
58600ad
165557d
f9406ed
3e41d81
1abce31
261d120
18db375
e8eb6c7
c646585
fbb6006
246a1b5
31e9b2d
1997e48
59ad921
a6b1fa7
7a334a9
83c7488
7aa12c5
551f2f5
1f0a840
f801a7a
0823e0f
b0d1dc9
32bff3a
0676758
a82ff74
c5c2516
b4184a3
613d0d1
fa1cd98
5b4e28e
fdf3706
debdd54
056279e
87de7aa
e0cfec2
1e22903
9816374
600f074
fb1fb77
13a632b
2f97d57
a61a483
8a5af82
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ActivityStreams hold activities, not Notes. This should be wrapped in a Create activity with the Note being the
object
.e.g. here's how we link to notes in the staticpub example: https://github.com/RangerMauve/staticpub.mauve.moe/blob/default/outbox.jsonld#L13
We may need to store the notes in a separate store and add a new
/:actor/note/:id
URL to resolve them.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see a requirement in the spec that states that OrderedCollections must hold Activitys. Actually, I was mostly copying off of Sutty's Jekyll implementation, which has plain Updates inside the outbox:
Am I getting something wrong?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, notice how the update you linked wraps the note in an Update activity. We should do the same here, but with a Create.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OH lol i thought Update was similar to a Note. But the ID there is just a link to the post, so do we need an endpoint for the individual Note?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might, lets see if just wrapping is enough for now though.