From 568b79427d892fa2f7cca6b0a9b9c8d2eb48376c Mon Sep 17 00:00:00 2001
From: Gabriele Dal Cengio <gabriele.dalcengio@gov.bc.ca>
Date: Tue, 17 Sep 2024 08:57:13 -0700
Subject: [PATCH] chore: Wrap routes in suspense wrapper that haven't done so
 in child data components

---
 .../industry_user/contacts/[contactId]/page.tsx           | 8 +++++++-
 .../app/bceidbusiness/industry_user/contacts/page.tsx     | 8 +++++++-
 .../app/bceidbusiness/industry_user/my-operator/page.tsx  | 8 +++++++-
 .../[operationId]/facilities/add-facility/page.tsx        | 8 +++++++-
 .../industry_user/operations/[operationId]/page.tsx       | 8 +++++++-
 .../industry_user_admin/contacts/[contactId]/page.tsx     | 8 +++++++-
 .../industry_user_admin/contacts/add-contact/page.tsx     | 8 +++++++-
 .../bceidbusiness/industry_user_admin/contacts/page.tsx   | 8 +++++++-
 .../industry_user_admin/my-operator/page.tsx              | 8 +++++++-
 .../[operationId]/facilities/add-facility/page.tsx        | 8 +++++++-
 .../industry_user_admin/operations/[operationId]/page.tsx | 8 +++++++-
 .../app/idir/cas_admin/contacts/[contactId]/page.tsx      | 8 +++++++-
 .../administration/app/idir/cas_admin/contacts/page.tsx   | 8 +++++++-
 .../app/idir/cas_analyst/contacts/[contactId]/page.tsx    | 8 +++++++-
 .../administration/app/idir/cas_analyst/contacts/page.tsx | 8 +++++++-
 .../reports/[version_id]/review-operator-data/page.tsx    | 8 +++++++-
 .../src/app/bceidbusiness/industry_user/reports/page.tsx  | 8 +++++++-
 .../reports/[version_id]/review-operator-data/page.tsx    | 8 +++++++-
 .../bceidbusiness/industry_user_admin/reports/page.tsx    | 8 +++++++-
 19 files changed, 133 insertions(+), 19 deletions(-)

diff --git a/bciers/apps/administration/app/bceidbusiness/industry_user/contacts/[contactId]/page.tsx b/bciers/apps/administration/app/bceidbusiness/industry_user/contacts/[contactId]/page.tsx
index 761658790d..139334426d 100644
--- a/bciers/apps/administration/app/bceidbusiness/industry_user/contacts/[contactId]/page.tsx
+++ b/bciers/apps/administration/app/bceidbusiness/industry_user/contacts/[contactId]/page.tsx
@@ -1,7 +1,13 @@
 import ContactPage from "@/administration/app/components/contacts/ContactPage";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default function Page({
   params: { contactId },
 }: Readonly<{ params: { contactId: string } }>) {
-  return <ContactPage contactId={contactId} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <ContactPage contactId={contactId} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/administration/app/bceidbusiness/industry_user/contacts/page.tsx b/bciers/apps/administration/app/bceidbusiness/industry_user/contacts/page.tsx
index a0a4aab808..35c9cdd523 100644
--- a/bciers/apps/administration/app/bceidbusiness/industry_user/contacts/page.tsx
+++ b/bciers/apps/administration/app/bceidbusiness/industry_user/contacts/page.tsx
@@ -1,11 +1,17 @@
 // 🚩 flagging that for shared routes between roles, "Page" code is a component for code maintainability
 import { ContactsSearchParams } from "@/administration/app/components/contacts/types";
 import ContactsPage from "@/administration/app/components/contacts/ContactsPage";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default async function Page({
   searchParams,
 }: Readonly<{
   searchParams: ContactsSearchParams;
 }>) {
-  return <ContactsPage searchParams={searchParams} isExternalUser={true} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <ContactsPage searchParams={searchParams} isExternalUser={true} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/administration/app/bceidbusiness/industry_user/my-operator/page.tsx b/bciers/apps/administration/app/bceidbusiness/industry_user/my-operator/page.tsx
index 4606c9258a..10c19fc361 100644
--- a/bciers/apps/administration/app/bceidbusiness/industry_user/my-operator/page.tsx
+++ b/bciers/apps/administration/app/bceidbusiness/industry_user/my-operator/page.tsx
@@ -1,7 +1,13 @@
 // 🚩 flagging that for shared routes between roles, `Operator` code is a component for code maintainability
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 import Operator from "@/administration/app/components/operators/Operator";
 
 export default async function Page() {
-  return <Operator />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <Operator />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/administration/app/bceidbusiness/industry_user/operations/[operationId]/facilities/add-facility/page.tsx b/bciers/apps/administration/app/bceidbusiness/industry_user/operations/[operationId]/facilities/add-facility/page.tsx
index 2dfe061209..ac4a078664 100644
--- a/bciers/apps/administration/app/bceidbusiness/industry_user/operations/[operationId]/facilities/add-facility/page.tsx
+++ b/bciers/apps/administration/app/bceidbusiness/industry_user/operations/[operationId]/facilities/add-facility/page.tsx
@@ -1,10 +1,16 @@
 import { UUID } from "crypto";
 import FacilityPage from "@/administration/app/components/facilities/FacilityPage";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default function Page({
   params,
 }: {
   params: Readonly<{ operationId: UUID }>;
 }) {
-  return <FacilityPage operationId={params.operationId} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <FacilityPage operationId={params.operationId} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/administration/app/bceidbusiness/industry_user/operations/[operationId]/page.tsx b/bciers/apps/administration/app/bceidbusiness/industry_user/operations/[operationId]/page.tsx
index 29beaf1ce3..077946fdc1 100644
--- a/bciers/apps/administration/app/bceidbusiness/industry_user/operations/[operationId]/page.tsx
+++ b/bciers/apps/administration/app/bceidbusiness/industry_user/operations/[operationId]/page.tsx
@@ -1,10 +1,16 @@
 import OperationInformationPage from "@/administration/app/components/operations/OperationInformationPage";
 import { UUID } from "crypto";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default async function Page({
   params: { operationId },
 }: Readonly<{
   params: { operationId: UUID };
 }>) {
-  return <OperationInformationPage operationId={operationId} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <OperationInformationPage operationId={operationId} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/administration/app/bceidbusiness/industry_user_admin/contacts/[contactId]/page.tsx b/bciers/apps/administration/app/bceidbusiness/industry_user_admin/contacts/[contactId]/page.tsx
index 761658790d..139334426d 100644
--- a/bciers/apps/administration/app/bceidbusiness/industry_user_admin/contacts/[contactId]/page.tsx
+++ b/bciers/apps/administration/app/bceidbusiness/industry_user_admin/contacts/[contactId]/page.tsx
@@ -1,7 +1,13 @@
 import ContactPage from "@/administration/app/components/contacts/ContactPage";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default function Page({
   params: { contactId },
 }: Readonly<{ params: { contactId: string } }>) {
-  return <ContactPage contactId={contactId} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <ContactPage contactId={contactId} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/administration/app/bceidbusiness/industry_user_admin/contacts/add-contact/page.tsx b/bciers/apps/administration/app/bceidbusiness/industry_user_admin/contacts/add-contact/page.tsx
index 22fa0548d3..c837184a2b 100644
--- a/bciers/apps/administration/app/bceidbusiness/industry_user_admin/contacts/add-contact/page.tsx
+++ b/bciers/apps/administration/app/bceidbusiness/industry_user_admin/contacts/add-contact/page.tsx
@@ -1,5 +1,11 @@
 import ContactPage from "@/administration/app/components/contacts/ContactPage";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default function Page() {
-  return <ContactPage />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <ContactPage />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/administration/app/bceidbusiness/industry_user_admin/contacts/page.tsx b/bciers/apps/administration/app/bceidbusiness/industry_user_admin/contacts/page.tsx
index a0a4aab808..35c9cdd523 100644
--- a/bciers/apps/administration/app/bceidbusiness/industry_user_admin/contacts/page.tsx
+++ b/bciers/apps/administration/app/bceidbusiness/industry_user_admin/contacts/page.tsx
@@ -1,11 +1,17 @@
 // 🚩 flagging that for shared routes between roles, "Page" code is a component for code maintainability
 import { ContactsSearchParams } from "@/administration/app/components/contacts/types";
 import ContactsPage from "@/administration/app/components/contacts/ContactsPage";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default async function Page({
   searchParams,
 }: Readonly<{
   searchParams: ContactsSearchParams;
 }>) {
-  return <ContactsPage searchParams={searchParams} isExternalUser={true} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <ContactsPage searchParams={searchParams} isExternalUser={true} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/administration/app/bceidbusiness/industry_user_admin/my-operator/page.tsx b/bciers/apps/administration/app/bceidbusiness/industry_user_admin/my-operator/page.tsx
index 4606c9258a..494c1adef8 100644
--- a/bciers/apps/administration/app/bceidbusiness/industry_user_admin/my-operator/page.tsx
+++ b/bciers/apps/administration/app/bceidbusiness/industry_user_admin/my-operator/page.tsx
@@ -1,7 +1,13 @@
 // 🚩 flagging that for shared routes between roles, `Operator` code is a component for code maintainability
 
 import Operator from "@/administration/app/components/operators/Operator";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default async function Page() {
-  return <Operator />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <Operator />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/administration/app/bceidbusiness/industry_user_admin/operations/[operationId]/facilities/add-facility/page.tsx b/bciers/apps/administration/app/bceidbusiness/industry_user_admin/operations/[operationId]/facilities/add-facility/page.tsx
index 2dfe061209..ac4a078664 100644
--- a/bciers/apps/administration/app/bceidbusiness/industry_user_admin/operations/[operationId]/facilities/add-facility/page.tsx
+++ b/bciers/apps/administration/app/bceidbusiness/industry_user_admin/operations/[operationId]/facilities/add-facility/page.tsx
@@ -1,10 +1,16 @@
 import { UUID } from "crypto";
 import FacilityPage from "@/administration/app/components/facilities/FacilityPage";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default function Page({
   params,
 }: {
   params: Readonly<{ operationId: UUID }>;
 }) {
-  return <FacilityPage operationId={params.operationId} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <FacilityPage operationId={params.operationId} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/administration/app/bceidbusiness/industry_user_admin/operations/[operationId]/page.tsx b/bciers/apps/administration/app/bceidbusiness/industry_user_admin/operations/[operationId]/page.tsx
index 29beaf1ce3..077946fdc1 100644
--- a/bciers/apps/administration/app/bceidbusiness/industry_user_admin/operations/[operationId]/page.tsx
+++ b/bciers/apps/administration/app/bceidbusiness/industry_user_admin/operations/[operationId]/page.tsx
@@ -1,10 +1,16 @@
 import OperationInformationPage from "@/administration/app/components/operations/OperationInformationPage";
 import { UUID } from "crypto";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default async function Page({
   params: { operationId },
 }: Readonly<{
   params: { operationId: UUID };
 }>) {
-  return <OperationInformationPage operationId={operationId} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <OperationInformationPage operationId={operationId} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/administration/app/idir/cas_admin/contacts/[contactId]/page.tsx b/bciers/apps/administration/app/idir/cas_admin/contacts/[contactId]/page.tsx
index 761658790d..139334426d 100644
--- a/bciers/apps/administration/app/idir/cas_admin/contacts/[contactId]/page.tsx
+++ b/bciers/apps/administration/app/idir/cas_admin/contacts/[contactId]/page.tsx
@@ -1,7 +1,13 @@
 import ContactPage from "@/administration/app/components/contacts/ContactPage";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default function Page({
   params: { contactId },
 }: Readonly<{ params: { contactId: string } }>) {
-  return <ContactPage contactId={contactId} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <ContactPage contactId={contactId} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/administration/app/idir/cas_admin/contacts/page.tsx b/bciers/apps/administration/app/idir/cas_admin/contacts/page.tsx
index 8d65d535f2..5e95278c50 100644
--- a/bciers/apps/administration/app/idir/cas_admin/contacts/page.tsx
+++ b/bciers/apps/administration/app/idir/cas_admin/contacts/page.tsx
@@ -1,11 +1,17 @@
 // 🚩 flagging that for shared routes between roles, "Page" code is a component for code maintainability
 import { ContactsSearchParams } from "@/administration/app/components/contacts/types";
 import ContactsPage from "@/administration/app/components/contacts/ContactsPage";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default async function Page({
   searchParams,
 }: Readonly<{
   searchParams: ContactsSearchParams;
 }>) {
-  return <ContactsPage searchParams={searchParams} isExternalUser={false} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <ContactsPage searchParams={searchParams} isExternalUser={false} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/administration/app/idir/cas_analyst/contacts/[contactId]/page.tsx b/bciers/apps/administration/app/idir/cas_analyst/contacts/[contactId]/page.tsx
index eba11a5203..9375027a02 100644
--- a/bciers/apps/administration/app/idir/cas_analyst/contacts/[contactId]/page.tsx
+++ b/bciers/apps/administration/app/idir/cas_analyst/contacts/[contactId]/page.tsx
@@ -1,7 +1,13 @@
 import ContactPage from "apps/administration/app/components/contacts/ContactPage";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default function Page({
   params: { contactId },
 }: Readonly<{ params: { contactId: string } }>) {
-  return <ContactPage contactId={contactId} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <ContactPage contactId={contactId} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/administration/app/idir/cas_analyst/contacts/page.tsx b/bciers/apps/administration/app/idir/cas_analyst/contacts/page.tsx
index 8d65d535f2..5e95278c50 100644
--- a/bciers/apps/administration/app/idir/cas_analyst/contacts/page.tsx
+++ b/bciers/apps/administration/app/idir/cas_analyst/contacts/page.tsx
@@ -1,11 +1,17 @@
 // 🚩 flagging that for shared routes between roles, "Page" code is a component for code maintainability
 import { ContactsSearchParams } from "@/administration/app/components/contacts/types";
 import ContactsPage from "@/administration/app/components/contacts/ContactsPage";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default async function Page({
   searchParams,
 }: Readonly<{
   searchParams: ContactsSearchParams;
 }>) {
-  return <ContactsPage searchParams={searchParams} isExternalUser={false} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <ContactsPage searchParams={searchParams} isExternalUser={false} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/reporting/src/app/bceidbusiness/industry_user/reports/[version_id]/review-operator-data/page.tsx b/bciers/apps/reporting/src/app/bceidbusiness/industry_user/reports/[version_id]/review-operator-data/page.tsx
index 4d3c8a806c..b5d4e4aff3 100644
--- a/bciers/apps/reporting/src/app/bceidbusiness/industry_user/reports/[version_id]/review-operator-data/page.tsx
+++ b/bciers/apps/reporting/src/app/bceidbusiness/industry_user/reports/[version_id]/review-operator-data/page.tsx
@@ -1,9 +1,15 @@
 import OperationReviewFormData from "../../../../../components/operations/OperationReviewFormData";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default async function Page({
   params,
 }: {
   params: { version_id: number };
 }) {
-  return <OperationReviewFormData version_id={params.version_id} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <OperationReviewFormData version_id={params.version_id} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/reporting/src/app/bceidbusiness/industry_user/reports/page.tsx b/bciers/apps/reporting/src/app/bceidbusiness/industry_user/reports/page.tsx
index d8203f4096..097b0601ca 100644
--- a/bciers/apps/reporting/src/app/bceidbusiness/industry_user/reports/page.tsx
+++ b/bciers/apps/reporting/src/app/bceidbusiness/industry_user/reports/page.tsx
@@ -1,11 +1,17 @@
 // 🚩 flagging that for shared routes between roles, "Page" code is a component for code maintainability
 import { OperationsSearchParams } from "@reporting/src/app/components/operations/types";
 import OperationsPage from "../../../components/routes/operations/Page";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default async function Page({
   searchParams,
 }: {
   searchParams: OperationsSearchParams;
 }) {
-  return <OperationsPage searchParams={searchParams} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <OperationsPage searchParams={searchParams} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/reporting/src/app/bceidbusiness/industry_user_admin/reports/[version_id]/review-operator-data/page.tsx b/bciers/apps/reporting/src/app/bceidbusiness/industry_user_admin/reports/[version_id]/review-operator-data/page.tsx
index 4d3c8a806c..b5d4e4aff3 100644
--- a/bciers/apps/reporting/src/app/bceidbusiness/industry_user_admin/reports/[version_id]/review-operator-data/page.tsx
+++ b/bciers/apps/reporting/src/app/bceidbusiness/industry_user_admin/reports/[version_id]/review-operator-data/page.tsx
@@ -1,9 +1,15 @@
 import OperationReviewFormData from "../../../../../components/operations/OperationReviewFormData";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default async function Page({
   params,
 }: {
   params: { version_id: number };
 }) {
-  return <OperationReviewFormData version_id={params.version_id} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <OperationReviewFormData version_id={params.version_id} />
+    </Suspense>
+  );
 }
diff --git a/bciers/apps/reporting/src/app/bceidbusiness/industry_user_admin/reports/page.tsx b/bciers/apps/reporting/src/app/bceidbusiness/industry_user_admin/reports/page.tsx
index d8203f4096..097b0601ca 100644
--- a/bciers/apps/reporting/src/app/bceidbusiness/industry_user_admin/reports/page.tsx
+++ b/bciers/apps/reporting/src/app/bceidbusiness/industry_user_admin/reports/page.tsx
@@ -1,11 +1,17 @@
 // 🚩 flagging that for shared routes between roles, "Page" code is a component for code maintainability
 import { OperationsSearchParams } from "@reporting/src/app/components/operations/types";
 import OperationsPage from "../../../components/routes/operations/Page";
+import { Suspense } from "react";
+import Loading from "@bciers/components/loading/SkeletonForm";
 
 export default async function Page({
   searchParams,
 }: {
   searchParams: OperationsSearchParams;
 }) {
-  return <OperationsPage searchParams={searchParams} />;
+  return (
+    <Suspense fallback={<Loading />}>
+      <OperationsPage searchParams={searchParams} />
+    </Suspense>
+  );
 }