1495424 - Lithuanian Play Store description for Firefox uses Beta strings

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Rimas Kudelis [use needinfo to ping]]

Attachment: beta strings in release.png

https://play.google.com/store/apps/details?id=org.mozilla.firefox&hl=lt offers a stable version of Firefox, but promotes it as beta. I've checked a couple other languages, but only found this issue with Lithuanian. It could be because some release strings were missing, but still I don't think they should have been inherited from Beta like this.

For the record, I've also translated all missing appstores strings into Lithuanian.

Comment 1

[Johan Lorenzo [:jlorenzo]]

Thank you for filing this bug! I think the root cause is bug 1444391 + some bad timing.

Strings on Google Play are updated every time we publish a new version. org.mozilla.firefox was last released on Sept 21st[1]. Automation downloads all available strings from [2] every time we kick a release off. In this, it was a day earlier[3]. In [3], the list of available locales didn't contain the 'lt' language. I see it stores_l10n does make it available these days[4]. Therefore, I think the lt locale will be updated next time a release is published. 

Why is it showing "beta" instead? Bug 1444391 made automation pick the locales from beta, instead of release about 6 months ago. My understanding is that the beta strings were available back then, but not the release ones. That's why it may have remained this way for the past 6 months.

Rimas, could you tell whether you updated the strings after Sept 21st? Could you also tell me if the locale was validated after this date?


[1] https://tools.taskcluster.net/groups/SNsKr_JsS6CRFjXmwP1z3w/tasks/UvKYGw5cTA2gKiIJ1m3knw/runs/0/logs/public%2Flogs%2Flive_backing.log#L448
[2] https://l10n.mozilla-community.org/stores_l10n/
[3] https://tools.taskcluster.net/groups/Ce1YswzfShKpUJ9EVU6J5Q/tasks/YIYwIDvLSwGQISeeEEXh-w/runs/0/logs/public%2Flogs%2Flive.log#L21
[4] https://l10n.mozilla-community.org/stores_l10n/api/v1/fx_android/listing/release/

Comment 2

[Johan Lorenzo [:jlorenzo]]

In the meantime, Sylvestre, do you mind running [1]? This should fix the existing strings until the next Fennec release comes up.

[1] https://github.com/mozilla-releng/mozapkpublisher/blob/1d9c44d2732a1d260b75e4fef1babcad4e0eb0d4/mozapkpublisher/update_apk_description.py

Comment 3

[Rimas Kudelis [use needinfo to ping]]

Johan, thanks for the explanation! I think I updated the strings on the weekend of 22nd-23rd, so just after the release happened.

Comment 4

[Sylvestre Ledru [:Sylvestre]]

Done!
Looks like it is fixed!

Comment 5

[Sylvestre Ledru [:Sylvestre]]

Attachment: image.png

Comment 6

[Delphine Lebédel [:delphine - Use needinfo]]

Merci Sylvestre et Johan ! :D

Comment 7

[Rimas Kudelis [use needinfo to ping]]

Yep, looks fixed. Merci beacoup!

Comment 8

[Delphine Lebédel [:delphine - Use needinfo]]

Ok closing then. Thanks everyone