fix: don't render application until we are sure the locale is initial…

…ized (#525)
dhis2 · Mar 2, 2021 · 4329e8e · 4329e8e
1 parent 1aba628
commit 4329e8e
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/adapter/src/components/AuthBoundary.js b/adapter/src/components/AuthBoundary.js
@@ -13,9 +13,11 @@ const settingsQuery = {
 
 export const AuthBoundary = ({ url, children }) => {
  const { loading, error, data } = useDataQuery(settingsQuery)
- useLocale(data && data.userSettings.keyUiLocale)
+ const locale = useLocale(
+ data && (data.userSettings.keyUiLocale || window.navigator.language)
+ )
 
- if (loading) {
+ if (loading || !locale) {
  return <LoadingMask />
  }
 

diff --git a/adapter/src/utils/useLocale.js b/adapter/src/utils/useLocale.js
@@ -1,6 +1,6 @@
 import i18n from '@dhis2/d2-i18n'
 import moment from 'moment'
-import { useEffect } from 'react'
+import { useState, useEffect } from 'react'
 
 i18n.setDefaultNamespace('default')
 
@@ -27,7 +27,13 @@ const setGlobalLocale = locale => {
 }
 
 export const useLocale = locale => {
+ const [result, setResult] = useState(undefined)
  useEffect(() => {
- setGlobalLocale(locale || window.navigator.language)
+ if (!locale) {
+ return
+ }
+ setGlobalLocale(locale)
+ setResult(locale)
  }, [locale])
+ return result
 }