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
show Development language again when development mode is enabled #19628
Conversation
It looks like you could otherwise leave that code as it was and just add "dev" to global config file, then that function would remove it for prod installs (though it would be incorrectly named, should be Is it possible it was previously just added by developers in config, and this change now means it is added automatically when you've enabled development mode? It might also need some documentation update to avoid future confusion. |
@justinvelluppillai You are right that seems a lot simpler (and I guess I lost that entry with #17930). So we could just add the entry for all users as it will be ignored for non-development anyway. |
If you don't want this PR to be closed automatically in 28 days then you need to assign the label 'Do not close'. |
I think it's actually better to only add the language in the code if development mode is enabled, rather than adding |
2bc9df3
to
1ab1d3b
Compare
Yep seems like a good approach to me. |
Description:
I am not sure if this broke with my changes and the weblate migration or before, but I can't see any place in
getAvailableLanguages()
dev
is added to the list.Review