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

mediaUpload: only post to outbox if it's a Create activity? #239

Closed
puckipedia opened this issue Jul 3, 2017 · 7 comments

Comments

@puckipedia
Copy link

commented Jul 3, 2017

mediaUpload, when posting any object, also posts it into the outbox. This is an issue, because you can only upload one file at a time. So you can't make e.g. a 4-picture post?

Per IRC:

<puckipedia> maybe if it's a Create activity, post it into the outbox, else just give it an ID?
<cwebber2> so you'd need an extra boolean in the form
<cwebber2> puckipedia: so make a {Video} as opposed to {Create: {Video}} ?
<puckipedia> well, if you send it as {Create: {Video}} it's post it into the outbox. if you send {Video} it only gives it an ID
<cwebber2> right, the id of the Video only
<cwebber2> but not post it wrapped in the Create to the outbox
@cwebber

This comment has been minimized.

Copy link
Collaborator

commented Jul 3, 2017

I'll get this integrated. Thanks for raising the issue @puckipedia :)

@strugee

This comment has been minimized.

Copy link

commented Jul 18, 2017

Linking to https://www.w3.org/wiki/Socialwg/2017-07-11-minutes#ActivityPub for posterity where we discussed a lot of things that didn't get captured in this issue.

@cwebber

This comment has been minimized.

Copy link
Collaborator

commented Oct 10, 2017

I'm starting to lean towards us dropping mediaUpload, since it's not clear we'll get the semantics right in time and it's at risk?

@cwebber

This comment has been minimized.

Copy link
Collaborator

commented Oct 10, 2017

we can always do as an extension

@cwebber

This comment has been minimized.

Copy link
Collaborator

commented Oct 10, 2017

<eprodrom> RESOLVED: Resolve https://github.com/w3c/activitypub/issues/239 by                      
           removing mediaUpload and specified behavior from ActivityPub spec                       
           proper, move to extension via SocialCG process
@cwebber

This comment has been minimized.

Copy link
Collaborator

commented Oct 10, 2017

I will make a new document proposing behavior and bring to the SocialCG tomorrow.

@rhiaro

This comment has been minimized.

Copy link
Member

commented Oct 24, 2017

I dumped the text from AP to https://www.w3.org/wiki/SocialCG/ActivityPub/MediaUpload for refinement by the CG (and added the uploadMedia property to https://www.w3.org/wiki/ActivityPub_extensions#uploadMedia).

@rhiaro rhiaro closed this in fae6a56 Oct 24, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.