Closed Bug 876457 Opened 12 years ago Closed 12 years ago

Implement S3 redirects for /edit and /remix routes in popcorn maker

Categories

(Webmaker Graveyard :: Popcorn Maker, defect)

x86_64
Linux
defect
Not set
blocker

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: thecount, Assigned: thecount)

References

Details

(Whiteboard: s=2013w24 p=1)

Attachments

(1 file)

For whatever the project's url is, a /edit should take you to the edit. Right now we're storing the remixUrl in the makeapi. I should be able to do makeResult.url + "/edit" or something similar. Thimble is going to be the same so it'll be tool agnostic.
Blocks: 876451
Blocks: 876459
Assignee: nobody → schranz.m
Summary: standardize popcorn's /edit url → Implement S3 redirects for /edit and /remix routes in popcorn maker
I think I need to be more clear on my intent in this ticket. This is, to create something at the user domain level. That is, inside S3, that allows other tools to generate an edit url from the domain level S3 URL. Also, something like this for thimble is bug 876440 Apparently that had to be done in two tickets too. So split this one as needed.
Assignee: schranz.m → scott
Depends on: 876522
It has some blockers before you can see the redirect being useful, but it does in fact redirect to where it is going to be, once the blockers land.
Attachment #758201 - Flags: review?(jon)
Not to point out the obvious, but this doesn't provide a local solution at all. Infact, the app crashes completely when using local storage.
yeah, I should make sure it works the old way if not using s3.
Whiteboard: s=2013w24
Whiteboard: s=2013w24 → s=2013w24 p=1
Severity: normal → blocker
Attachment #758201 - Flags: review- → review?(jon)
Attachment #758201 - Flags: review?(jon) → review?(david.humphrey)
Comment on attachment 758201 [details] [review] https://github.com/mozilla/popcorn.webmaker.org/pull/34 Put this back on me for review after Jon's server.js rewrite lands, and you've rebased.
Attachment #758201 - Flags: review?(david.humphrey)
Depends on: 863366
Attachment #758201 - Flags: review?(jon)
Attachment #758201 - Flags: review?(jon) → review+
Staged: https://github.com/mozilla/popcorn.webmaker.org/commit/366d61b434a47ad52f4ffe592ab94d97cb006a91 It does not work until jbuck plugs in the rest of his changes.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Attachment mime type: text/plain → text/x-github-pull-request
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: