Closed Bug 569560 Opened 14 years ago Closed 14 years ago

Add redirect for s.m.c/contribute

Categories

(support.mozilla.org :: General, defect, P1)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: djst, Assigned: jsocol)

References

()

Details

Attachments

(1 file)

We are working on a paper flyer to distribute at the LinuxTag event (Wednesday June 9th) and need a redirect for support.mozilla.com/contribute to point to support.mozilla.com/%locale%/kb/How+to+contribute.

Don't quote me on this, but in an attempt to make this easy for webdev, I think what needs to be added near the end of http://svn.mozilla.org/projects/sumo/trunk/webroot/htaccess.dist is:

RewriteRule ^([\-a-zA-Z]+)/contribute$                  /$1/contribute/ [R]
...
RewriteRule ^contribute\/?$               tiki-index.php?page=How+to+contribute   [QSA,L]
RewriteRule ^([\-a-zA-Z]+)/contribute\/?$               tiki-index.php?page=How+to+contribute&locale=$1   [QSA,L]
Close, but not quite. I'll pick this up today.
Assignee: nobody → james
Target Milestone: --- → 1.5.5
Can this redirect? IE: Can the URL change once people navigate to /contribute, or does it need to stay unchanged?
Yes, it can definitely redirect. It should work the same way as the /forum redirect, which expands to https://support.mozilla.com/%detected_locale%/kb/Support+Website+Forums. Thanks!
Attached patch really simpleSplinter Review
This leaves the locale-detection to Tiki and just maps /contribute to /kb/How+to+contribute. Simple.
Attachment #449039 - Flags: review?(paulc)
Comment on attachment 449039 [details] [diff] [review]
really simple

WFM.
Attachment #449039 - Flags: review?(paulc) → review+
r68277.

QA/David: This just makes /contribute redirect to /kb/How+to+contribute, and Tiki figures out the language from your browser. Sound right?
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Sounds perfect. 

Btw, when I looked at htaccess.dist, I saw at least three lines of code per similar redirect (/forum, /chat) -- that's why I proposed that code in comment 0. Just flagging it in case we're missing something, though I bet you know what you're doing. ;)
And indeed, this things works on stage.
(And my grammer is badder than normal)
Verified FIXED; tested |en-US| and |de| in my accept-language prefs.
Status: RESOLVED → VERIFIED
in-testsuite+: bug 570245
Flags: in-testsuite? → in-testsuite+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: