-
Notifications
You must be signed in to change notification settings - Fork 184
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
(feat) Add support for setting default language via html lang attribute #695
Conversation
Size Change: +127 B (0%) Total Size: 2.13 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we change default-language
with default-locale to match with the global setting default-locale
?
Thanks for working on this @ibacher ! Really grateful 🙏
@vasharma05 Good call! |
@@ -87,6 +89,7 @@ export async function runBuild(args: BuildArgs) { | |||
env: "production", | |||
apiUrl: buildConfig.apiUrl || args.apiUrl, | |||
configUrls: configUrls, | |||
defaultLocale: args.defaultLocale || buildConfig.defaultLocale, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: we should update the other properties to follow this pattern, i.e., explicitly CLI args should always override the configuration file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Requirements
For changes to apps
If applicable
Summary
Adds support for configuring a default language via the build config or command line arguments.
While this is technically duplicative of the
default_locale
system setting from the backend, we have good reason to ensure that some kind of default language is set in the frontend in case the backend is, for whatever reason unavailable.@vasharma05 To support Mekom's use-case, once merged, I'll add an overridable environment variable to the O3 RefApp so that this can be set on a per-instance basis.
Screenshots
Related Issue
https://issues.openmrs.org/browse/O3-2155
Other