903216 - Some l10n pages show {OK} from *.lang files

Status: VERIFIED FIXED

(www.mozilla.org :: L10N, defect)

Description

[Hector Zhao [:hectorz]]

In bug 745716, lowercase {ok} is stripped, but there're some uppercase {OK} used in *.lang files. Examples:

http://www.mozilla.org/uk/firefox/new/
http://www.mozilla.org/uk/foundation/annualreport/2011/
http://www.mozilla.org/sk/firefox/new/
http://www.mozilla.org/zh-TW/tabzilla/tabzilla.js
http://www.mozilla.org/te/firefox/new/

PR coming soon

Comment 1

[Hector Zhao [:hectorz]]

https://github.com/mozilla/bedrock/pull/1143

Comment 2

[[github robot]]

Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/60252c0ad15d6c25ebff2c75fc58b485894592b3
Bug 903216: Some l10n pages show {OK} from *.lang files

https://github.com/mozilla/bedrock/commit/e4ee8d8bc9cefa818e3bc73687fa86f5769128bb
Merge pull request #1143 from l-hedgehog/bug-903216-case-insensitive-ok

Bug 903216: Some l10n pages show {OK} from *.lang files

Comment 3

[Paul [:pmac] McLanahan]

For QA:

1. Go to http://www.mozilla.org/uk/foundation/annualreport/2011/
2. Search on the page for {OK}. I see it after "Firefox Flicks".
    a. The "{OK}" should not appear. It is to indicate that the string has been dealt with even though it matches the en-US string.
4. Go to https://www-dev.allizom.org/uk/foundation/annualreport/2011/
5. See that "Firefox Flicks" does not have "{OK}" after it.

Comment 4

[raymond [:retornam] (inactive)]

fixed on stage https://www.allizom.org/uk/foundation/annualreport/2011/

Comment 5

[raymond [:retornam] (inactive)]

fixed on production