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

added Greek and Arabic supported locales in staging #4419

Conversation

zenmonkeykstop
Copy link
Contributor

@zenmonkeykstop zenmonkeykstop commented May 8, 2019

Status

Ready for review

Description of Changes

Fixes #4418.

Adds Greek and Arabic as supported locales in staging environment, to simplify verification of locale-specific features and layouts.

Testing

  • with this branch
make build-debs
make staging
  • then, verify that the source and journalist interface on the staging environment contain the locale dropdowns, with the choice of English, Greek, and Arabic.
  • for extra credit, run the testinfra tests for your container of choice with molecule verify -s {container}-staging and verify that the locale tests still pass.

Deployment

Staging-only change.

Checklist

If you made changes to the system configuration:

  • Configuration tests pass - **there are unrelated testinfra failures (at least on the qubes-staging scenario) that will be fixed separately. **

If you made non-trivial code changes:

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

@@ -36,14 +36,16 @@ def test_securedrop_application_apt_dependencies(host, package):

def test_securedrop_application_test_locale(host):
"""
Ensure SecureDrop DEFAULT_LOCALE is present.
Ensure both SecureDrop DEFAULT_LOCALE and SUPPORTED_LOCALES are present.`
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: trailing ` at end of line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed.

@zenmonkeykstop zenmonkeykstop force-pushed the 4418-add-locales-in-staging branch from fe17c56 to e51f745 Compare May 9, 2019 02:54
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.

good suggestion! checked it works as advertised in staging, new testinfra test passes, good to merge when CI passes

@redshiftzero redshiftzero merged commit 13e3a0f into freedomofpress:develop May 9, 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.

Staging environments should be configured with multiple locales enabled.
2 participants