Pass the isDark param when redirecting to the account theme

codegouvfr · Dec 11, 2024 · 240d05e · 240d05e
1 parent ac5320b
commit 240d05e
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 2 deletions.
diff --git a/web/src/core/bootstrap.ts b/web/src/core/bootstrap.ts
@@ -22,6 +22,7 @@ type ParamsOfBootstrapCore = {
         termsOfServiceUrl: LocalizedString<Language>;
     }) => string;
     getCurrentLang: () => Language;
+    getIsDark: () => boolean;
     onMoved: (params: { redirectUrl: string }) => void;
 };
 

diff --git a/web/src/core/usecases/userAccountManagement/thunks.ts b/web/src/core/usecases/userAccountManagement/thunks.ts
@@ -130,7 +130,7 @@ export const thunks = {
                 ,
                 getState,
                 {
-                    paramsOfBootstrapCore: { getCurrentLang }
+                    paramsOfBootstrapCore: { getCurrentLang, getIsDark }
                 }
             ] = args;
 
@@ -162,6 +162,18 @@ export const thunks = {
                 url = newUrl;
             }
 
+            {
+
+                const { newUrl } = addParamToUrl({
+                    url,
+                    "name": "dark",
+                    "value": `${getIsDark()}`
+                });
+
+                url = newUrl;
+
+            }
+
             return url;
         }
 } satisfies Thunks;
diff --git a/web/src/ui/App.tsx b/web/src/ui/App.tsx
@@ -66,7 +66,11 @@ const { CoreProvider } = createCoreProvider({
         );
 
         window.location.href = newUrl.toString();
-    }
+    },
+    // NOTE: Passed so that it can be injected in the Account management URL.  
+    // I'm not comfortable with this level of indirection, this is only UI related logic
+    // that shouldn't involve the core. However I do it this way for consistency sake. 
+    getIsDark
 });
 
 export default function App() {