Closed Bug 441689 Opened 16 years ago Closed 16 years ago

/firefox/ is a 404 on mozillaonline.com

Categories

(Websites :: www.mozillaonline.com, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: clouserw, Assigned: clouserw)

Details

Attachments

(1 file, 1 obsolete file)

To reproduce:

1) load http://mozillaonline.com/
2) Click on the giant "Firefox 3" at the top

Since I'm merging stuff for bug 440922 I'll handle this but needed a bug for people
This is fixed on trunk and production.  I changed the link to /products/firefox/.

Didn't we have a discussion about keeping URLs consistent and just making this /firefox/?  Is that what broke the link in the first place?  I'm thinking just putting in a rewrite for /firefox/ to /products/firefox/ might be a good idea.
No, we didn't, :(. I don't know about this. And I think I have changed all /firefox/ to /products/firefox/ in the other pages.

A rewrite statement would be a great idea.
Attached patch rewrite (obsolete) — Splinter Review
here's a rewrite
Attachment #327052 - Flags: review?(reed)
Comment on attachment 327052 [details] [diff] [review]
rewrite

>+RewriteRule ^firefox(.*)$ /products/firefox$1 [PT]

That could possibly match firefoxflicks or something random like that...

How about: RewriteRule ^firefox(/.*)?$ /products/firefox$1
Attachment #327052 - Flags: review?(reed) → review-
Comment on attachment 327052 [details] [diff] [review]
rewrite

with the [PT] at the end, of course.
Attached patch new rewriteSplinter Review
wfm
Attachment #327052 - Attachment is obsolete: true
Attachment #327053 - Flags: review?(reed)
Comment on attachment 327053 [details] [diff] [review]
new rewrite

r=me
Attachment #327053 - Flags: review?(reed) → review+
on trunk and prod.  thanks.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Component: Other → www.mozillaonline.com
QA Contact: other → www-mozillaonline-com
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: