Add redirects for new Webmaker IA

RESOLVED FIXED

Status

Webmaker
webmaker.org
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: secretrobotron, Assigned: akirose)

Tracking

Details

(Whiteboard: [ia][dec12])

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
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.
(Reporter)

Updated

3 years ago
Blocks: 1104272

Comment 1

3 years ago
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
Flags: needinfo?(secretrobotron)
Flags: needinfo?(jon)
(Reporter)

Updated

3 years ago
Status: NEW → ASSIGNED
(Reporter)

Comment 2

3 years ago
Oh! Yeah. Doing the redirect within the app makes sense.
Keeping this open atm in case there is more that needs attention.
Flags: needinfo?(secretrobotron)

Comment 3

3 years ago
Yes, lets do it within the webmaker.org app so we can test it.
Flags: needinfo?(jon)
(Reporter)

Comment 4

3 years ago
Looks like gallery is the only one for now. Assigning to Aki since she's in the build.
Assignee: johns → aki
(Assignee)

Comment 5

3 years ago
Created attachment 8535826 [details] [review]
https://github.com/mozilla/webmaker.org/pull/1034
Attachment #8535826 - Flags: review?(jon)

Comment 6

3 years ago
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.
Attachment #8535826 - Flags: review?(jon) → review-
(Assignee)

Comment 7

3 years ago
Yeah I'm pretty sure that's what we're supposed to be doing? Bobby?
Flags: needinfo?(bobby)
(Reporter)

Comment 8

3 years ago
The page should actually be /explore.
/gallery should redirect to /explore.

Cool?
Flags: needinfo?(bobby)
(Assignee)

Updated

3 years ago
Attachment #8535826 - Flags: review- → review?(jon)

Updated

3 years ago
Attachment #8535826 - Flags: review?(jon) → review+
(Assignee)

Updated

3 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED

Comment 9

3 years ago
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
You need to log in before you can comment on or make changes to this bug.