1108804 - Add redirects for new Webmaker IA

Status: RESOLVED FIXED

(Webmaker Graveyard :: webmaker.org, defect)

Description

[Bobby Richter [:secretrobotron]]

Fairly short list so far:
* https://webmaker.org/gallery -> https://webmaker.org/explore

Should these be 301s?

Will add more as I think of them.

Comment 1

[JP Schneider :jp]

301 indeed.

However, we typically aren't redirecting off of paths, and if we do, we usually do it from the app itself (in this case webmaker.org on github)

What do you want to do? :D

Comment 2

[Bobby Richter [:secretrobotron]]

Oh! Yeah. Doing the redirect within the app makes sense.
Keeping this open atm in case there is more that needs attention.

Comment 3

[Jon Buckley [:jbuck]]

Yes, lets do it within the webmaker.org app so we can test it.

Comment 4

[Bobby Richter [:secretrobotron]]

Looks like gallery is the only one for now. Assigning to Aki since she's in the build.

Comment 5

[Aki Rose Braun [:akirose]]

Attachment: https://github.com/mozilla/webmaker.org/pull/1034

Comment 6

[Jon Buckley [:jbuck]]

Comment on attachment 8535826 [details] [review]
https://github.com/mozilla/webmaker.org/pull/1034

I don't think this is correct -- it's redirecting /explore to /gallery.

Comment 7

[Aki Rose Braun [:akirose]]

Yeah I'm pretty sure that's what we're supposed to be doing? Bobby?

Comment 8

[Bobby Richter [:secretrobotron]]

The page should actually be /explore.
/gallery should redirect to /explore.

Cool?

Comment 9

[[github robot]]

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

https://github.com/mozilla/webmaker.org/commit/f8023f9435fd86edf0f88cf2beb3604db8b00c2a
Fixes Bug 1108804 - Add redirects for new Webmaker IA