851207 - Use canonical casing for Serbian locale script subtags (sr-Cyrl and sr-Latn)

Status: RESOLVED FIXED

(support.mozilla.org :: Localization, task, P3)

Description

[Gordon P. Hemsley [:GPHemsley]]

The Serbian locales on SUMO use non-canonical casing for the script subtag in their locale codes: 'sr-CYRL' and 'sr-LATN'. (I assume this was based on legacy assumptions at some point that should have been restricted to region subtags.)

While not strictly important, I think it would be better in the long run for these locales to use the canonical casing of 'sr-Cyrl' and 'sr-Latn'. (The canonical casing for script subtags is for the first letter to be uppercase and the rest to be lowercase.)

Comment 1

[Ricky Rosario [:rrosario, :r1cky]]

I think we are ok with changing this in settings.py and lib/languages.json and being done. I don't *think* we have to migrate locale strings in the db:

In [3]: Document.objects.filter(locale='en-us').count()
Out[3]: 1144

In [4]: Document.objects.filter(locale='en-US').count()
Out[4]: 1144

In [5]: Document.objects.filter(locale='sr-Cyrl').count()
Out[5]: 63

In [6]: Document.objects.filter(locale='sr-cyrl').count()
Out[6]: 63

Comment 2

[Ricky Rosario [:rrosario, :r1cky]]

Based on Comment 1, this should be an almost trivial change.

Comment 3

[Ricky Rosario [:rrosario, :r1cky]]

In a pull request:
https://github.com/mozilla/kitsune/pull/1249

Comment 4

[Ricky Rosario [:rrosario, :r1cky]]

Landed on master:
https://github.com/mozilla/kitsune/commit/ac0b3bd6a7ac0bb03709e7b99c59cc3e42a72b2b

Deployed to prod now.