Fix: Setting config.default_locale / CONFIG_DEFAULT_LOCALE updates default locale of the site #578
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This pull requests includes a fix for the non-working configuration option
config.default_locale
/CONFIG_DEFAULT_LOCALE
.Setting the value now properly applies the locale for any unregistered user visiting the site.
Visiting the site as a registered user still applies the locale set in the user's profile.
The fix has been tested by using configuration value set in the environment variables of the docker-compose.yml
Example for
CONFIG_DEFAULT_LOCALE="de"
What type of PR is this? (check all applicable)
Related Tickets & Documents
Fixes #577
Screenshots/Recordings
Before fix: before-fix_locale-all
After fix:
Steps to QA
CONFIG_DEFAULT_LOCALE
to "de" using configuration in docker-compose.yml (see above)docker-compose up --build
(Note: Only parts of the translation file are currently translated)