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

Status: RESOLVED FIXED

(Webmaker Graveyard :: Popcorn Maker, defect)

Description

[Scott [:thecount] Downe]

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.

Comment 1

[Scott [:thecount] Downe]

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.

Comment 2

[Scott [:thecount] Downe]

Attachment: https://github.com/mozilla/popcorn.webmaker.org/pull/34

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.

Comment 3

[Matthew Schranz [:mjschranz]]

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.

Comment 4

[Scott [:thecount] Downe]

yeah, I should make sure it works the old way if not using s3.

Comment 5

[Jon Buckley [:jbuck]]

Comment on attachment 758201 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/34

r-, comments in the PR

Comment 6

[Scott [:thecount] Downe]

Comment on attachment 758201 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/34

Updates.

Pretty much just re did it.

Comment 7

[David Humphrey (:humph)]

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.

Comment 8

[Scott [:thecount] Downe]

Staged: https://github.com/mozilla/popcorn.webmaker.org/commit/366d61b434a47ad52f4ffe592ab94d97cb006a91

It does not work until jbuck plugs in the rest of his changes.