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

Implement ForgeFed vocabulary. #136

Merged
merged 1 commit into from
Jul 5, 2020
Merged

Conversation

BenLubar
Copy link
Contributor

@BenLubar BenLubar commented May 9, 2020

Caveat: the "description" property is probably wrong; it should match the definition of the ActivityPub(!) "source" property which is currently not in go-fed.

Caveat: the "description" property is probably wrong; it should match the definition of the ActivityPub(!) "source" property which is currently not in go-fed.
@cjslep
Copy link
Member

cjslep commented May 26, 2020

Do you mind adding the "source" property to the activitypub spec? I think it was an oversight on my part, and once that is reviewed, I can easily double-check the "description" property here.

If I don't hear back in a day or two, I'll make an attempt at supporting the "source" property.

@cjslep
Copy link
Member

cjslep commented Jul 5, 2020

I'm going to tackle adding the source property now. Once it is in we can compare notes with the description property. I'd like to get this change in before cutting the v1.0.0 tag.

@cjslep
Copy link
Member

cjslep commented Jul 5, 2020

FYI, 2d143b2 contains the shiny new source property.

@cjslep cjslep merged commit 2da4dc1 into go-fed:master Jul 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants