From 4329e8e37ab4155038b0145bcb0c4bbbc8cc9312 Mon Sep 17 00:00:00 2001
From: Austin McGee <947888+amcgee@users.noreply.github.com>
Date: Tue, 2 Mar 2021 03:23:22 +0100
Subject: [PATCH] fix: don't render application until we are sure the locale is
 initialized (#525)

---
 adapter/src/components/AuthBoundary.js |  6 ++++--
 adapter/src/utils/useLocale.js         | 10 ++++++++--
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/adapter/src/components/AuthBoundary.js b/adapter/src/components/AuthBoundary.js
index 26243e387..d91c2b9aa 100644
--- 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
index f9a0df6cd..075ba81cb 100644
--- 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
 }