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
 }