Closed
Bug 840416
Opened 13 years ago
Closed 13 years ago
Redirect www.mozilla.org/add-ons/kodakcd to AMO
Categories
(www.mozilla.org :: General, defect)
www.mozilla.org
General
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: cmore, Assigned: pmac)
Details
(Whiteboard: r=112915 b=trunk)
Attachments
(1 file)
|
789 bytes,
patch
|
Details | Diff | Splinter Review |
One of top 404 on Mozilla.org is pointing to:
www.mozilla.org/add-ons/kodakcd
In the .htaccess file on line 239, there is a kodak redirect.
http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/.htaccess?view=markup
Can we change the redirect to something like this?
RewriteRule ^en-US/add-ons/kodak.*$ https://addons.mozilla.org/en-US/firefox/addon/4441 [R=permanent]
That should match kodak|kodak/|kodakcd|kodakcd/ or really anything after kodak. If this doesn't make sense, we should just do a new redirect for kodakcd to the homepage of addons.mozilla.org.
| Reporter | ||
Comment 1•13 years ago
|
||
:pmac / :sancus Do you have any thoughts on this redirect?
Comment 2•13 years ago
|
||
The page was removed by Bug 634320 without setting up any redirect. According to the SVN changelog, it was the CD distro version of the Kodak EasyShare Gallery Companion landing page.
http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/en-US/add-ons/kodakcd/index.html?view=log&pathrev=87450
Because the content was mostly the same as the non-CD version, it makes sense to add the redirect, IMO.
http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/en-US/add-ons/kodak/index.html?revision=15942&view=co&pathrev=69207
http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/en-US/add-ons/kodakcd/index.html?revision=79510&view=co&pathrev=87450
OS: Mac OS X → All
Hardware: x86 → All
| Assignee | ||
Comment 4•13 years ago
|
||
If this is really only for en-US, then I think this is good.
Attachment #712936 -
Flags: review?(chrismore.bugzilla)
| Reporter | ||
Comment 5•13 years ago
|
||
That looks fine to me. Yes, it is only en-US and there are many variations after the string "kodak" and thus your regex will match the rest fine.
| Assignee | ||
Updated•13 years ago
|
Whiteboard: r=112915 b=trunk
| Assignee | ||
Comment 6•13 years ago
|
||
Committed to prod in r112917.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
| Reporter | ||
Updated•13 years ago
|
Attachment #712936 -
Flags: review?(chrismore.bugzilla) → review?
Comment 7•13 years ago
|
||
verified fixed www.mozilla.org/add-ons/kodakcd ---> https://addons.mozilla.org/en-US/firefox/addon/kodak-easyshare-gallery-compan/
Status: RESOLVED → VERIFIED
| Assignee | ||
Updated•13 years ago
|
Attachment #712936 -
Flags: review?
You need to log in
before you can comment on or make changes to this bug.
Description
•