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

AS2 vocab translation coordination/funkiness #116

Open
1 of 4 tasks
strugee opened this issue Nov 22, 2017 · 2 comments
Open
1 of 4 tasks

AS2 vocab translation coordination/funkiness #116

strugee opened this issue Nov 22, 2017 · 2 comments

Comments

@strugee
Copy link

strugee commented Nov 22, 2017

Heya!

I'm currently working on translating AS1 vocabulary to AS2 in pump.io in preparation for our migration to ActivityPub - I wanted to see what Granary does and try to align if possible.

Looking at https://github.com/snarfed/granary/blob/d76e923c2fb3142223668c665c3bff7f76207882/granary/as2.py I spotted a couple things that seemed funky:

Let me know if I missed something. pump-io/pump.io#1483 (comment) is tracking this in pump.io FWIW.

Cheers :)

@snarfed
Copy link
Owner

snarfed commented Nov 24, 2017

thanks for looking at this! i expect you're right on these points in general. I'll fix them soon!

@snarfed
Copy link
Owner

snarfed commented Nov 26, 2017

Shouldn't
type = obj.pop('type', None)
be @type, not type?

AS2 prefers type, not @type, right? @type is JSON-LD, right? https://w3c.github.io/activitystreams/core/ only shows @type for JSON-LD compatibility, and https://w3c.github.io/activitypub/ doesn't mention @type at all.

snarfed added a commit that referenced this issue Nov 27, 2017
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

No branches or pull requests

2 participants