Closed Bug 572056 Opened 14 years ago Closed 14 years ago

About Us bookmark is always a 301

Categories

(Firefox :: Bookmarks & History, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 4.0b1

People

(Reporter: abuchanan, Assigned: mbrubeck)

Details

Attachments

(1 file, 1 obsolete file)

mozilla.com has a permanent rewrite rule for the About Us bookmark.  A more better URL is http://www.mozilla.com/about/
Status: NEW → ASSIGNED
Component: General → Bookmarks & History
Product: Fennec → Firefox
QA Contact: general → bookmarks
Attached patch patch (obsolete) — Splinter Review
This is not actually in the default Fennec bookmarks - it must have been synced from a desktop Firefox profile using Firefox Sync.  This patch updates the default Firefox bookmark to use the newer URL.

See bug 537183 for more information/context.
Assignee: nobody → mbrubeck
Attachment #451393 - Flags: review?(mak77)
Attached patch patch v2Splinter Review
Minor update: Forgot to update a localization-note comment.
Attachment #451393 - Attachment is obsolete: true
Attachment #451399 - Flags: review?(mak77)
Attachment #451393 - Flags: review?(mak77)
Comment on attachment 451399 [details] [diff] [review]
patch v2

Did you check that other addresses in that file are still fine as of today?

It should be fine, I've not found any test specifically using this uri.
Attachment #451399 - Flags: review?(mak77) → review+
(In reply to comment #3)
> Did you check that other addresses in that file are still fine as of today?

No, I didn't check any other addresses (and I wouldn't know which ones are good).  Maybe abuchanan can double-check http://mxr.mozilla.org/mozilla-central/source/browser/locales/generic/profile/bookmarks.html.in for any other problems?
The rest look good to me
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/7657b6ee181a
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a6
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: