Skip to content
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

Add Czech to list of supported languages #4885

Merged
merged 3 commits into from
Oct 10, 2019

Conversation

rmol
Copy link
Contributor

@rmol rmol commented Oct 8, 2019

Status

Ready for review

Description of Changes

Fixes #4882.

This adds Czech to the list of supported languages.

Also updates the translation-test script so it gets the list of supported locales from i18n_tool.py and will no longer have to be updated when we change that list.

Testing

  • Ensure that the Czech language code (cs) is included in the SUPPORTED_LOCALES variable in securedrop/config.py. You can edit the file to add it, or simply delete the file and run make test-config.
  • Run make dev and verify that Czech is an option in the language menu:

image

Deployment

This adds a new supported language. To offer it, sites will need to add it via securedrop-admin sdconfig, then run securedrop-admin install.

Checklist

If you made changes to the server application code:

  • Linting (make lint) and tests (make test) pass in the development container

If you made non-trivial code changes:

  • I have written a test plan and validated it for this PR

rmol added 2 commits October 8, 2019 20:02
contributors:
  1000101
  Michal Stanke
  darmozrac
  michaela-bot

updated from:
  repo: https://github.com/freedomofpress/securedrop-i18n
  revision: 570f297
Also update the translation-test script so it gets the list of
supported locales from i18n_tool.py and will no longer have to be
updated when we change that list.
Copy link
Contributor

@redshiftzero redshiftzero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just the docs change and the one question inline and then this PR lgtm!

@rmol rmol requested a review from zenmonkeykstop as a code owner October 9, 2019 18:39
@redshiftzero redshiftzero merged commit bca26dd into freedomofpress:develop Oct 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Czech to list of supported languages
2 participants