From 73a898703b5ac719db1e3461491c08dd53417069 Mon Sep 17 00:00:00 2001
From: Courtney Myers <courtney.myers@erg.com>
Date: Fri, 22 Jul 2022 15:15:05 -0400
Subject: [PATCH 1/3] Move client app's checking of csb enrollment closed
 status to get response from /api/epa-data, so client app doesn't need to be
 rebuilt for env variable to take effect

---
 app/client/.env.example                  | 12 +++++-------
 app/client/src/components/dashboard.tsx  |  5 +++--
 app/client/src/config.tsx                | 11 +----------
 app/client/src/contexts/user.tsx         |  1 +
 app/client/src/routes/allRebates.tsx     |  8 +++++---
 app/client/src/routes/existingRebate.tsx |  5 +++--
 app/client/src/routes/newRebate.tsx      |  6 +++---
 app/server/app/routes/api.js             |  1 +
 8 files changed, 22 insertions(+), 27 deletions(-)

diff --git a/app/client/.env.example b/app/client/.env.example
index d2e849f0..4d4ad206 100644
--- a/app/client/.env.example
+++ b/app/client/.env.example
@@ -1,8 +1,6 @@
-# NOTE: For local development, set `REACT_APP_CSB_ENROLLMENT_PERIOD`,
-# `REACT_APP_FORMIO_BASE_URL`, and `REACT_APP_FORMIO_PROJECT_NAME` to the same
-# values used in the server app's `CSB_ENROLLMENT_PERIOD`, `FORMIO_BASE_URL`,
-# and `FORMIO_PROJECT_NAME` env variables. When deployed, a GitHub Actions
-# Workflow does this in the "build" npm script.
-REACT_APP_CSB_ENROLLMENT_PERIOD=
+# NOTE: For local development, set `REACT_APP_FORMIO_BASE_URL` and
+# `REACT_APP_FORMIO_PROJECT_NAME` to the same values used in the server app's
+# `FORMIO_BASE_URL` and `FORMIO_PROJECT_NAME` env variables. When deployed, a
+# GitHub Actions Workflow does this in the "build" npm script.
 REACT_APP_FORMIO_BASE_URL=
-REACT_APP_FORMIO_PROJECT_NAME=
+REACT_APP_FORMIO_PROJECT_NAME=
\ No newline at end of file
diff --git a/app/client/src/components/dashboard.tsx b/app/client/src/components/dashboard.tsx
index 45f876e0..89a78c72 100644
--- a/app/client/src/components/dashboard.tsx
+++ b/app/client/src/components/dashboard.tsx
@@ -8,7 +8,6 @@ import icons from "uswds/img/sprite.svg";
 import {
   serverUrl,
   serverUrlForLinks,
-  enrollmentClosed,
   formioBaseUrl,
   formioProjectUrl,
   fetchData,
@@ -190,7 +189,9 @@ export default function Dashboard() {
             </Link>
           )}
 
-          {pathname.startsWith("/rebate") || enrollmentClosed ? (
+          {pathname.startsWith("/rebate") ||
+          (epaUserData.status === "success" &&
+            epaUserData.data?.enrollmentClosed) ? (
             <button
               className="margin-bottom-1 usa-button font-sans-2xs"
               disabled
diff --git a/app/client/src/config.tsx b/app/client/src/config.tsx
index 3e8ad33e..d5c2bd47 100644
--- a/app/client/src/config.tsx
+++ b/app/client/src/config.tsx
@@ -2,17 +2,10 @@ const {
   NODE_ENV,
   REACT_APP_SERVER_BASE_PATH,
   REACT_APP_CLOUD_SPACE,
-  REACT_APP_CSB_ENROLLMENT_PERIOD,
   REACT_APP_FORMIO_BASE_URL,
   REACT_APP_FORMIO_PROJECT_NAME,
 } = process.env;
 
-if (!REACT_APP_CSB_ENROLLMENT_PERIOD) {
-  throw new Error(
-    "Required REACT_APP_CSB_ENROLLMENT_PERIOD environment variable not found."
-  );
-}
-
 if (!REACT_APP_FORMIO_BASE_URL) {
   throw new Error(
     "Required REACT_APP_FORMIO_BASE_URL environment variable not found."
@@ -52,8 +45,6 @@ export const serverUrlForLinks =
 export const cloudSpace =
   NODE_ENV === "development" ? "dev" : REACT_APP_CLOUD_SPACE || "";
 
-export const enrollmentClosed = REACT_APP_CSB_ENROLLMENT_PERIOD === "closed";
-
 export const formioBaseUrl = REACT_APP_FORMIO_BASE_URL;
 
 const formioProjectName = REACT_APP_FORMIO_PROJECT_NAME;
@@ -75,7 +66,7 @@ export const messages = {
   timeout:
     "For security reasons, you have been logged out due to 15 minutes of inactivity.",
   logout: "You have successfully logged out.",
-  csbEnrollmentClosed: "The CSB enrollment period is closed.",
+  enrollmentClosed: "The CSB enrollment period is closed.",
 };
 
 /**
diff --git a/app/client/src/contexts/user.tsx b/app/client/src/contexts/user.tsx
index 9b10c1ef..c59e968a 100644
--- a/app/client/src/contexts/user.tsx
+++ b/app/client/src/contexts/user.tsx
@@ -11,6 +11,7 @@ type Props = {
 };
 
 type EpaUserData = {
+  enrollmentClosed: boolean;
   mail: string;
   memberof: string;
   exp: number;
diff --git a/app/client/src/routes/allRebates.tsx b/app/client/src/routes/allRebates.tsx
index 7b1c598e..dde0ae85 100644
--- a/app/client/src/routes/allRebates.tsx
+++ b/app/client/src/routes/allRebates.tsx
@@ -2,7 +2,7 @@ import { useEffect } from "react";
 import { Link } from "react-router-dom";
 import icons from "uswds/img/sprite.svg";
 // ---
-import { serverUrl, enrollmentClosed, fetchData, messages } from "../config";
+import { serverUrl, fetchData, messages } from "../config";
 import Loading from "components/loading";
 import Message from "components/message";
 import MarkdownContent from "components/markdownContent";
@@ -13,7 +13,7 @@ import { useFormsState, useFormsDispatch } from "contexts/forms";
 
 export default function AllRebates() {
   const { content } = useContentState();
-  const { samUserData } = useUserState();
+  const { epaUserData, samUserData } = useUserState();
   const { rebateFormSubmissions } = useFormsState();
   const dispatch = useFormsDispatch();
 
@@ -138,7 +138,9 @@ export default function AllRebates() {
                     <tr
                       key={_id}
                       className={
-                        state === "submitted" || enrollmentClosed
+                        state === "submitted" ||
+                        (epaUserData.status === "success" &&
+                          epaUserData.data?.enrollmentClosed)
                           ? "text-italic text-base-dark"
                           : ""
                       }
diff --git a/app/client/src/routes/existingRebate.tsx b/app/client/src/routes/existingRebate.tsx
index 89712659..0480f38a 100644
--- a/app/client/src/routes/existingRebate.tsx
+++ b/app/client/src/routes/existingRebate.tsx
@@ -12,7 +12,7 @@ import { useNavigate, useParams } from "react-router-dom";
 import { Formio, Form } from "@formio/react";
 import { cloneDeep, isEqual } from "lodash";
 // ---
-import { serverUrl, enrollmentClosed, fetchData } from "../config";
+import { serverUrl, fetchData } from "../config";
 import { getUserInfo } from "../utilities";
 import Loading from "components/loading";
 import Message from "components/message";
@@ -393,7 +393,8 @@ function ExistingRebateContent() {
           }}
           options={{
             readOnly:
-              submissionData.state === "submitted" || enrollmentClosed
+              submissionData.state === "submitted" ||
+              epaUserData.data.enrollmentClosed
                 ? true
                 : false,
             noAlerts: true,
diff --git a/app/client/src/routes/newRebate.tsx b/app/client/src/routes/newRebate.tsx
index 2ae0aad1..52e4543b 100644
--- a/app/client/src/routes/newRebate.tsx
+++ b/app/client/src/routes/newRebate.tsx
@@ -3,7 +3,7 @@ import { useNavigate } from "react-router-dom";
 import { DialogOverlay, DialogContent } from "@reach/dialog";
 import icons from "uswds/img/sprite.svg";
 // ---
-import { serverUrl, enrollmentClosed, messages, fetchData } from "../config";
+import { serverUrl, messages, fetchData } from "../config";
 import { getUserInfo } from "../utilities";
 import Loading from "components/loading";
 import Message from "components/message";
@@ -73,8 +73,8 @@ export default function NewRebate() {
         >
           <div className="usa-modal__content">
             <div className="usa-modal__main">
-              {enrollmentClosed ? (
-                <Message type="info" text={messages.csbEnrollmentClosed} />
+              {epaUserData.data.enrollmentClosed ? (
+                <Message type="info" text={messages.enrollmentClosed} />
               ) : activeSamRecords.length <= 0 ? (
                 <Message type="info" text={messages.samNoResults} />
               ) : (
diff --git a/app/server/app/routes/api.js b/app/server/app/routes/api.js
index 621f1db5..f8dade8e 100644
--- a/app/server/app/routes/api.js
+++ b/app/server/app/routes/api.js
@@ -97,6 +97,7 @@ router.get("/helpdesk-access", ensureHelpdesk, (req, res) => {
 router.get("/epa-data", (req, res) => {
   // Explicitly return only required attributes from user info
   res.json({
+    enrollmentClosed: process.env.CSB_ENROLLMENT_PERIOD === "closed",
     mail: req.user.mail,
     memberof: req.user.memberof,
     exp: req.user.exp,

From 55039c3913becad5e937d4e5efdffc42340255b5 Mon Sep 17 00:00:00 2001
From: Courtney Myers <courtney.myers@erg.com>
Date: Fri, 22 Jul 2022 15:16:11 -0400
Subject: [PATCH 2/3] Remove no longer needed client app env variable in npm
 build step of GitHub Actions workflows

---
 .github/workflows/dev.yml     | 1 -
 .github/workflows/proto.yml   | 1 -
 .github/workflows/staging.yml | 1 -
 3 files changed, 3 deletions(-)

diff --git a/.github/workflows/dev.yml b/.github/workflows/dev.yml
index 89aae36c..53d29b30 100644
--- a/.github/workflows/dev.yml
+++ b/.github/workflows/dev.yml
@@ -87,7 +87,6 @@ jobs:
           PUBLIC_URL="$SERVER_URL" \
             REACT_APP_SERVER_BASE_PATH="$SERVER_BASE_PATH" \
             REACT_APP_CLOUD_SPACE="$CLOUD_SPACE" \
-            REACT_APP_CSB_ENROLLMENT_PERIOD="$CSB_ENROLLMENT_PERIOD" \
             REACT_APP_FORMIO_BASE_URL="$FORMIO_BASE_URL" \
             REACT_APP_FORMIO_PROJECT_NAME="$FORMIO_PROJECT_NAME" \
             npm run build
diff --git a/.github/workflows/proto.yml b/.github/workflows/proto.yml
index 4be23c66..4fe4e16f 100644
--- a/.github/workflows/proto.yml
+++ b/.github/workflows/proto.yml
@@ -86,7 +86,6 @@ jobs:
           PUBLIC_URL="$SERVER_URL" \
             REACT_APP_SERVER_BASE_PATH="$SERVER_BASE_PATH" \
             REACT_APP_CLOUD_SPACE="$CLOUD_SPACE" \
-            REACT_APP_CSB_ENROLLMENT_PERIOD="$CSB_ENROLLMENT_PERIOD" \
             REACT_APP_FORMIO_BASE_URL="$FORMIO_BASE_URL" \
             REACT_APP_FORMIO_PROJECT_NAME="$FORMIO_PROJECT_NAME" \
             npm run build
diff --git a/.github/workflows/staging.yml b/.github/workflows/staging.yml
index 36c60c46..b7309b00 100644
--- a/.github/workflows/staging.yml
+++ b/.github/workflows/staging.yml
@@ -87,7 +87,6 @@ jobs:
           PUBLIC_URL="$SERVER_URL" \
             REACT_APP_SERVER_BASE_PATH="$SERVER_BASE_PATH" \
             REACT_APP_CLOUD_SPACE="$CLOUD_SPACE" \
-            REACT_APP_CSB_ENROLLMENT_PERIOD="$CSB_ENROLLMENT_PERIOD" \
             REACT_APP_FORMIO_BASE_URL="$FORMIO_BASE_URL" \
             REACT_APP_FORMIO_PROJECT_NAME="$FORMIO_PROJECT_NAME" \
             npm run build

From db59dd51dca8220ebb3158493bdf1839a62f8aac Mon Sep 17 00:00:00 2001
From: Courtney Myers <courtney.myers@erg.com>
Date: Fri, 22 Jul 2022 15:19:09 -0400
Subject: [PATCH 3/3] Add empty line at end of client app's example .env file

---
 app/client/.env.example | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/client/.env.example b/app/client/.env.example
index 4d4ad206..489e9e51 100644
--- a/app/client/.env.example
+++ b/app/client/.env.example
@@ -3,4 +3,4 @@
 # `FORMIO_BASE_URL` and `FORMIO_PROJECT_NAME` env variables. When deployed, a
 # GitHub Actions Workflow does this in the "build" npm script.
 REACT_APP_FORMIO_BASE_URL=
-REACT_APP_FORMIO_PROJECT_NAME=
\ No newline at end of file
+REACT_APP_FORMIO_PROJECT_NAME=