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

Merge i18n into master #172

Merged
merged 113 commits into from
Jun 15, 2021
Merged

Merge i18n into master #172

merged 113 commits into from
Jun 15, 2021

Conversation

wasade
Copy link
Member

@wasade wasade commented Jun 10, 2021

No description provided.

wasade and others added 30 commits May 4, 2021 16:34
Adds messages.pot and first usages of gettext
…go into es_mx. Updated links to be replaced by babel in jinja2
…'t hooked up to actually switch language anywhere though
…ld that is always mm/dd/yyyy. Replaces validation to localize the normalized field on init, and normalize the localized field on validation. Uses the jquery datepicker's i18n extensions for configuring the datepicker
…ge settings to ApiRequests so that we query surveys in the right language
ci/pip_requirements.txt Outdated Show resolved Hide resolved
microsetta_interface/implementation.py Outdated Show resolved Hide resolved
microsetta_interface/implementation.py Outdated Show resolved Hide resolved
microsetta_interface/implementation.py Outdated Show resolved Hide resolved
microsetta_interface/implementation.py Outdated Show resolved Hide resolved
<label for="language" name="language_label">{{ _('Preferred language for future communications') }}*</label>
<br>
<select id="language" name="language">
<option value="en_US" {% if account.language == "en_US" %} selected {% endif %} >English</option>
Copy link
Member Author

Choose a reason for hiding this comment

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

We should add "en_GB", see #175

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think push beyond this PR to avoid scope creep. Once #173 is in, this particular line will come for free with a change to model_i18n, but until we have a separate .PO file, the difference is pretty meaningless.

microsetta_interface/templates/new_results_page.jinja2 Outdated Show resolved Hide resolved
microsetta_interface/templates/new_results_page.jinja2 Outdated Show resolved Hide resolved
@wasade wasade merged commit 6ae3866 into master Jun 15, 2021
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.

4 participants