883838 - edit and remix paths don't resolve the proper urls

Status: RESOLVED FIXED

(Webmaker Graveyard :: webmaker.org, defect)

Description

[Scott [:thecount] Downe]

They are missing some specifics to work.

Shouldn't be too much though.

Need to add /edit and /remix to the actual paths.

Also, the actual redirect path seems to have an undefined in it.

Comment 1

[Scott [:thecount] Downe]

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

Testing this now that the rewrite is done, shows that a few things were missed.

Comment 2

[Jon Buckley [:jbuck]]

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

r-

app.locals gets mixed in with res.locals when you call res.render() or app.render(). I'd turn that html template into a view that you render

Comment 3

[[github robot]]

Commit pushed to master at https://github.com/mozilla/popcorn.webmaker.org

https://github.com/mozilla/popcorn.webmaker.org/commit/17e2287e6ca0799dc0a27c92aaeb9669898215a7
Merge pull request #57 from ScottDowne/t883838

Fix bug 883838 - Fixed remix and edit routes.