Closed
Bug 528572
Opened 15 years ago
Closed 15 years ago
Update product details for Firefox 3.6 Beta 3
Categories
(www.mozilla.org :: General, defect)
www.mozilla.org
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Pike, Assigned: pascalc)
Details
Attachments
(2 files)
12.84 KB,
patch
|
clouserw
:
review+
|
Details | Diff | Splinter Review |
791 bytes,
patch
|
clouserw
:
review+
|
Details | Diff | Splinter Review |
We're taking more localizations for Firefox 3.6 Beta 3. I'll be landing updates to shipped-locales and l10n-changesets_mozilla-1.9.2, filing this bug on www.mozilla.com right away to update product details based on that.
Flags: blocking-firefox3.6?
Reporter | ||
Comment 1•15 years ago
|
||
shipped-locale and l10n-changesets done, http://hg.mozilla.org/releases/mozilla-1.9.2/rev/5f8370a86737, http://hg.mozilla.org/build/buildbot-configs/rev/5f3524211d06. Over to websites staff, the locale list is http://hg.mozilla.org/releases/mozilla-1.9.2/file/5f8370a86737/browser/locales/shipped-locales. Added: cy, en-GB, eo, gu-IN, kk, kn, ko, mk, ml, nb-NO, nn-NO, si, sl, ta-LK
Summary: Update l10n-changesets and shipped-locales for Firefox 3.6 Beta 3 → Update product details for Firefox 3.6 Beta 3
Comment 2•15 years ago
|
||
Pascal, can we get a patch for the product-details?
Assignee | ||
Comment 3•15 years ago
|
||
What will be the release date? (for the FirefoxHistory class)
Assignee | ||
Comment 4•15 years ago
|
||
I am attaching the patch to product-details for review. Note that this page does not update the History file for releases.
Assignee: nobody → pascalc
Attachment #412283 -
Flags: review?(clouserw)
Comment 5•15 years ago
|
||
Comment on attachment 412283 [details] [diff] [review] product-details patch wfm, we should add the history when we push it
Attachment #412283 -
Flags: review?(clouserw) → review+
Comment 6•15 years ago
|
||
Please push this to trunk-stage; we're ready to start testing the download pages there.
Comment 8•15 years ago
|
||
Could we get these pushed live? Thanks!
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 9•15 years ago
|
||
I take that back. Hang on for now, please.
Comment 10•15 years ago
|
||
And this seems to have gone live, at the appropriate time, so re-closing.
Status: REOPENED → RESOLVED
Closed: 15 years ago → 15 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 11•15 years ago
|
||
... reopened. Dance. We're only showing 54 locales on all-beta.html, my suspect is that locales that only show in $beta_builds don't get picked up by all-beta.html. The LATEST_FIREFOX_DEVEL_VERSION of those probably needs to move up to $primary_builds. Guessing here only, though.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 12•15 years ago
|
||
I looked at the class and the method used getDownloadTableForDevelBuilds() calls the primary_builds array. I think that duplicating locales in both primary_builds and beta_builds is going to be error prone with our frequent stable/dev releases. I would prefer getDownloadTableForDevelBuilds to use both arrays, here is a patch for that. I can land it on trunk for testing today if needed.
Attachment #413060 -
Flags: review?(clouserw)
Comment 13•15 years ago
|
||
Comment on attachment 413060 [details] [diff] [review] patch firefoxDetails.class.php to call all builds in all-beta.html If that's what you would prefer, it's fine, but consistency would put another table underneath the primary one. Beta build isn't the same as beta locale.
Attachment #413060 -
Flags: review?(clouserw) → review+
Assignee | ||
Comment 14•15 years ago
|
||
ok, committed in r56315
Comment 15•15 years ago
|
||
If it works, we should push it live ASAP.
Comment 16•15 years ago
|
||
One locale's still missing, it's 60 in the list vs 62-1 (ja/ja-jp-mac) in the shipped-locales
Comment 17•15 years ago
|
||
...and the missing locale is "or"
Reporter | ||
Comment 18•15 years ago
|
||
Yes, that's missing in firefoxDetails, in the beta builds. Probably missed out on that for Beta 2 as well.
Assignee | ||
Comment 19•15 years ago
|
||
Oriya added in r56376 on trunk
Comment 20•15 years ago
|
||
Can we, uh, please fix this? Don't think it blocks the release, but needs immediate attention.
Flags: blocking-firefox3.6? → blocking-firefox3.6-
Comment 21•15 years ago
|
||
(In reply to comment #20) > Can we, uh, please fix this? Don't think it blocks the release, but needs > immediate attention. If QA or y'all give me the thumbs up, I'll tag a new product details for production
Looks good to me: ran a link checker (http://validator.w3.org/checklink?uri=https%3A%2F%2Fwww-trunk.stage.mozilla.com%2Fen-US%2Ffirefox%2Fall.html&hide_type=all&depth=&check=Check), and the only broken link was to one mirror (hopefully Sentry catches that?) It'd be nice if the QAE team could look over https://www-trunk.stage.mozilla.com/en-US/firefox/all.html too.
Reporter | ||
Comment 23•15 years ago
|
||
Will, seems like QA its deeds, can you push this live?
Comment 24•15 years ago
|
||
done in production, r56597
Status: REOPENED → RESOLVED
Closed: 15 years ago → 15 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Component: www.mozilla.org/firefox → www.mozilla.org
Updated•12 years ago
|
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in
before you can comment on or make changes to this bug.
Description
•