Fix: Address warnings of React Router

Update to react-router-dom 6.28.1 which will not raise warnings if the
future config setting of a router is set.

Enable the start transition setting for react router which should be
fine because we don't user React.lazy (yet).

Disable relative splat path because I am not sure if it will affect GSA.

package.json

@@ -65,7 +65,7 @@
     "react-dom": "^18.3.1",
     "react-i18next": "^15.4.0",
     "react-redux": "^9.1.2",
-    "react-router-dom": "^6.28.0",
+    "react-router-dom": "^6.28.1",
     "redux": "^5.0.1",
     "redux-logger": "^3.0.6",
     "resize-observer-polyfill": "^1.5.1",

src/web/routes.jsx

@@ -213,9 +213,14 @@ const LoggedInRoutes = () => {
 
             <Route element={<PageNotFound />} path="/notfound" />
             <Route element={<AuditReportsPage />} path="/auditreports" />
-            <Route element={<DeltaAuditReportDetailsPage />}
-                        path="/auditreport/delta/:id/:deltaid" />
-            <Route element= {<AuditReportDetailsPage />} path="/auditreport/:id" />
+            <Route
+              element={<DeltaAuditReportDetailsPage />}
+              path="/auditreport/delta/:id/:deltaid"
+            />
+            <Route
+              element={<AuditReportDetailsPage />}
+              path="/auditreport/:id"
+            />
             <Route element={<Navigate to="/dashboards" />} path="/" />
 
             <Route element={<PageNotFound />} path="*" />
@@ -241,7 +246,16 @@ const AppRoutes = () => {
   }
 
   return (
-    <Router>{isLoggedIn ? <LoggedInRoutes /> : <LoggedOutRoutes />}</Router>
+    <Router
+      future={{
+        // eslint-disable-next-line camelcase
+        v7_startTransition: true,
+        // eslint-disable-next-line camelcase
+        v7_relativeSplatPath: false,
+      }}
+    >
+      {isLoggedIn ? <LoggedInRoutes /> : <LoggedOutRoutes />}
+    </Router>
   );
 };
 

src/web/utils/testing.jsx

@@ -144,7 +144,17 @@ export const rendererWith = (
         <TestingLicenseProvider license={license}>
           <TestingStoreProvider store={store}>
             {router ? (
-              <BrowserRouter initialEntries={['/']}>{children}</BrowserRouter>
+              <BrowserRouter
+                future={{
+                  // eslint-disable-next-line camelcase
+                  v7_startTransition: true,
+                  // eslint-disable-next-line camelcase
+                  v7_relativeSplatPath: false,
+                }}
+                initialEntries={['/']}
+              >
+                {children}
+              </BrowserRouter>
             ) : (
               children
             )}