@autobusal/common 1.15.0 → 1.15.1

package/UnderConstruction/useUnderConstruction.ts

@@ -1,4 +1,3 @@
-import { useLocation } from 'react-router-dom';
 import { useGetSettings } from '@autobusal/providers/services';
 
 /**
@@ -7,18 +6,28 @@ import { useGetSettings } from '@autobusal/providers/services';
  * Edited: Ferjolt Ozuni - Date: 2026-08-01
  *
  * Without this the switch is one-way: the holding page replaces the whole
- * router, so /admin goes behind it too and the only way to turn maintenance
- * back OFF is a database edit. Found by actually switching it on and then
+ * router, so /admin went behind it too and the only way to turn maintenance
+ * back OFF was a database edit. Found by actually switching it on and then
  * trying to switch it off again.
  *
  * Staff routes are the exception, not the public site - somebody has to be
  * able to log in and finish the work the site is closed for. Everything a
- * visitor can reach still gets the holding page, including deep links.
+ * visitor can reach still gets the holding page, deep links included.
  */
 const ALWAYS_OPEN = [ '/admin', '/account', '/auth' ];
 
 /**
- * Whether the holding page should replace the app for the current route.
+ * Whether the holding page should replace the app right now.
+ *
+ * Reads the path from `window.location` rather than react-router's
+ * useLocation: this runs ABOVE <RouterProvider> (it decides whether to
+ * render the router at all), so there is no router context to read from -
+ * calling useLocation here throws and blanks the entire page, which is
+ * exactly what happened the first time.
+ *
+ * The consequence is that it does not re-evaluate on client-side
+ * navigation, which is fine: the holding page renders no links, so the only
+ * way to leave it is a real page load, and that re-runs this.
  *
  * obtapi sends `construction` only while the brand is actually under
  * construction, so its presence is the switch.
@@ -26,14 +35,14 @@ const ALWAYS_OPEN = [ '/admin', '/account', '/auth' ];
 const useUnderConstruction = (): boolean => {
   const { data: settings } = useGetSettings();
 
-  const location = useLocation();
-
   if (!settings?.construction) {
     return false;
   }
 
-  return !ALWAYS_OPEN.some(path => (
-    location.pathname === path || location.pathname.startsWith(`${ path }/`)
+  const path = typeof window === 'undefined' ? '/' : window.location.pathname;
+
+  return !ALWAYS_OPEN.some(open => (
+    path === open || path.startsWith(`${ open }/`)
   ));
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autobusal/common",
-  "version": "1.15.0",
+  "version": "1.15.1",
   "author": "Ferjolt Ozuni",
   "type": "module",
   "main": "index.ts"