-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move zh-tw and zh-cn locales to zh-hant and zh-hans, respectively #4515
Comments
tbh, this change (in django) makes me uneasy. |
See https://code.djangoproject.com/ticket/18419 for the discussion in Django. It looks like In any case, I'm not sure we'll be able to avoid the change, unless we keep a copy / symlink the old locales files for the strings translated in django itself and ignore the warning. |
In practice, The difference within the subsets are usually vocabularies and translation costumes (much like the difference between |
Re-reading the django code I don't think it's a blocker for us... yet. When django removes the backwards-compatibility code, we'll be in trouble, but at the moment it will still work - and produce a deprecation warning. |
According to the discussion in the ticket, there should be a fallback for The pull request django/django#1868 seems to have implemented the idea and was merged years ago. But the behaviour of https://addons.mozilla.org/ does not follow the default Django. So is there something in the middle that went wrong? |
We don't follow the default because Mozilla products don't - the source of truth for our languages is currently https://product-details.mozilla.org/1.0/languages.json and it doesn't support zh-hant/zh-hans. Right now, everything works - because of the fallback code. I created this issue because in more recent versions of Django, the fallback code emits a deprecation warning indicating that it might go away in the future. However, as I said in my previous comment, this is not necessarily a blocker for upgrading to Django 1.11, we can live with the deprecation warning, as long as the fallback code is still there. |
This may also be impacted by https://github.com/mozilla-services/cloudops-deployment/issues/1404 |
blocked by mozilla-services/cloudops-deployment#1404 pretty much |
not blocked by mozilla-services/cloudops-deployment#1404 any more. Looks like the fallback was removed in django1.9 @diox 🐼 |
This issue has been automatically marked as stale because it has not had recent activity. If you think this bug should stay open, please comment on the issue with further details. Thank you for your contributions. |
I don't think there is any improvement on the situation. Please keep this issue open. |
When language isn't specified for
You can find such default mapping rules ( Take Hong Kong for example, their government writes in Mandarin Chinese ( However, Cantonese ( By using the Likely Subtags data, you can get rid of chaos without having to learn the development history of the Chinese macro-language. |
This issue has been automatically marked as stale because it has not had recent activity. If you think this bug should stay open, please comment on the issue with further details. Thank you for your contributions. |
Recent version of django moved zh-tw and zh-cn locales to zh-hant and zh-hans. There is some code for backwards-compatibility, but that will go away and we'll need to handle that.
I had started doing that in mozilla/addons-server#5811 but it requires more thought so I reverted that commit and moved the discussion here. One of the issues is that if we change the locales like this, we need to add redirects from the old locales and migrate data in translated fields.
This blocks #4292(See #4515)The text was updated successfully, but these errors were encountered: