From 248f569f8e7c86b1f1c220cd40504b7783b2cacf Mon Sep 17 00:00:00 2001
From: "Maureen E. Zitouni" <maureen.zitouni@gmail.com>
Date: Wed, 27 Mar 2024 16:26:56 -0400
Subject: [PATCH 01/12] Fix bug - trying to sort a read-only property of array

---
 src/containers/TexterTodoList.jsx | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/containers/TexterTodoList.jsx b/src/containers/TexterTodoList.jsx
index 2280e30a2..8e97c9c47 100644
--- a/src/containers/TexterTodoList.jsx
+++ b/src/containers/TexterTodoList.jsx
@@ -58,7 +58,8 @@ class TexterTodoList extends React.Component {
   }
 
   renderTodoList(assignments) {
-    return assignments
+    const sortedAssignments = [...assignments];
+    return sortedAssignments
       .sort((x, y) => {
         // Sort with feedback at the top, and then based on Text assignment size
         const xHasFeedback =

From 71af4a73478a609ee9ddb256911491685576d213 Mon Sep 17 00:00:00 2001
From: "Maureen E. Zitouni" <maureen.zitouni@gmail.com>
Date: Thu, 28 Mar 2024 17:35:18 -0400
Subject: [PATCH 02/12] Fix spacing

---
 src/components/AssignmentTexter/Controls.jsx | 86 ++++++++++----------
 1 file changed, 43 insertions(+), 43 deletions(-)

diff --git a/src/components/AssignmentTexter/Controls.jsx b/src/components/AssignmentTexter/Controls.jsx
index 13b3700ea..b3376c36a 100644
--- a/src/components/AssignmentTexter/Controls.jsx
+++ b/src/components/AssignmentTexter/Controls.jsx
@@ -508,8 +508,8 @@ export class AssignmentTexterContactControls extends React.Component {
 
     const otherResponsesLink =
       currentInteractionStep &&
-        currentInteractionStep.question.filteredAnswerOptions.length > 6 &&
-        filteredCannedResponses.length ? (
+      currentInteractionStep.question.filteredAnswerOptions.length > 6 &&
+      filteredCannedResponses.length ? (
         <div className={css(flexStyles.popoverLink)} key={"otherresponses"}>
           <a
             href="#otherresponses"
@@ -522,8 +522,8 @@ export class AssignmentTexterContactControls extends React.Component {
 
     const searchBar = currentInteractionStep &&
       currentInteractionStep.question.answerOptions.length +
-      campaign.cannedResponses.length >
-      5 && (
+        campaign.cannedResponses.length >
+        5 && (
         <SearchBar
           onRequestSearch={this.handleSearchChange}
           onChange={this.handleSearchChange}
@@ -661,12 +661,12 @@ export class AssignmentTexterContactControls extends React.Component {
                   margin: "9px",
                   color:
                     this.state.optOutMessageText ===
-                      this.props.campaign.organization.optOutMessage
+                    this.props.campaign.organization.optOutMessage
                       ? "white"
                       : "#494949",
                   backgroundColor:
                     this.state.optOutMessageText ===
-                      this.props.campaign.organization.optOutMessage
+                    this.props.campaign.organization.optOutMessage
                       ? "#727272"
                       : "white"
                 }}
@@ -879,9 +879,9 @@ export class AssignmentTexterContactControls extends React.Component {
       shortCannedResponses = shortCannedResponses.filter(script => {
         var textLength = global.HIDE_BRANCHED_SCRIPTS
           ? this.getShortButtonText(
-            script.title,
-            cannedResponseScript ? 40 : 13
-          ).length
+              script.title,
+              cannedResponseScript ? 40 : 13
+            ).length
           : script.title.length;
 
         if (joinedLength + 1 + textLength < 80) {
@@ -965,7 +965,7 @@ export class AssignmentTexterContactControls extends React.Component {
       <div className={css(flexStyles.subButtonsExitButtons)}>
         <Button
           onClick={
-            !disabled ? this.handleOpenAnswerResponsePopover : noAction => { }
+            !disabled ? this.handleOpenAnswerResponsePopover : noAction => {}
           }
           style={{
             backgroundColor: this.props.muiTheme.palette.background.default,
@@ -1197,40 +1197,40 @@ export class AssignmentTexterContactControls extends React.Component {
     const content = firstMessage
       ? this.renderFirstMessage(enabledSideboxes)
       : [
-        this.renderToolbar(enabledSideboxes),
-        <div
-          key="superSectionMessagePage"
-          className={css(flexStyles.superSectionMessagePage)}
-        >
-          {this.state.contactListOpen &&
-            this.renderAssignmentContactsList(
-              this.props.assignment.contacts,
-              this.props.contact,
-              this.props.updateCurrentContactById
-            )}
-          <div className={css(flexStyles.superSectionMessageListAndControls)}>
-            <ContactToolbar
-              campaignContact={this.props.contact}
-              campaign={this.props.campaign}
-              navigationToolbarChildren={this.props.navigationToolbarChildren}
-              toggleContactList={this.toggleContactList}
-            />
-            {this.renderMessageBox(
-              <MessageList
-                contact={this.props.contact}
-                currentUser={this.props.currentUser}
-                messages={this.props.contact.messages}
-                organizationId={this.props.organizationId}
-                review={this.props.review}
-                styles={messageListStyles}
-                hideMedia={this.state.hideMedia}
-              />,
-              enabledSideboxes
-            )}
-            {this.renderMessageControls(enabledSideboxes)}
+          this.renderToolbar(enabledSideboxes),
+          <div
+            key="superSectionMessagePage"
+            className={css(flexStyles.superSectionMessagePage)}
+          >
+            {this.state.contactListOpen &&
+              this.renderAssignmentContactsList(
+                this.props.assignment.contacts,
+                this.props.contact,
+                this.props.updateCurrentContactById
+              )}
+            <div className={css(flexStyles.superSectionMessageListAndControls)}>
+              <ContactToolbar
+                campaignContact={this.props.contact}
+                campaign={this.props.campaign}
+                navigationToolbarChildren={this.props.navigationToolbarChildren}
+                toggleContactList={this.toggleContactList}
+              />
+              {this.renderMessageBox(
+                <MessageList
+                  contact={this.props.contact}
+                  currentUser={this.props.currentUser}
+                  messages={this.props.contact.messages}
+                  organizationId={this.props.organizationId}
+                  review={this.props.review}
+                  styles={messageListStyles}
+                  hideMedia={this.state.hideMedia}
+                />,
+                enabledSideboxes
+              )}
+              {this.renderMessageControls(enabledSideboxes)}
+            </div>
           </div>
-        </div>
-      ];
+        ];
     return (
       <div
         className={css(flexStyles.topContainer)}

From 2bc5bd47d1d18112658a2b8ed1341e1152fda534 Mon Sep 17 00:00:00 2001
From: "Maureen E. Zitouni" <maureen.zitouni@gmail.com>
Date: Fri, 29 Mar 2024 01:51:18 -0400
Subject: [PATCH 03/12] Fix error where copied nested objects are immutable

---
 src/components/AssignmentTexter/Controls.jsx |  6 +++++-
 src/components/utils.jsx                     | 13 +++++++++++++
 2 files changed, 18 insertions(+), 1 deletion(-)

diff --git a/src/components/AssignmentTexter/Controls.jsx b/src/components/AssignmentTexter/Controls.jsx
index b3376c36a..d1b88ce85 100644
--- a/src/components/AssignmentTexter/Controls.jsx
+++ b/src/components/AssignmentTexter/Controls.jsx
@@ -35,6 +35,7 @@ import {
 import { dataTest } from "../../lib/attributes";
 import ContactToolbar from "./ContactToolbar";
 import { getCookie, setCookie } from "../../lib/cookie";
+import { deepCopy } from "../utils";
 
 export class AssignmentTexterContactControls extends React.Component {
   constructor(props) {
@@ -124,7 +125,10 @@ export class AssignmentTexterContactControls extends React.Component {
     let currentInteractionStep = null;
 
     if (availableSteps.length > 0) {
-      currentInteractionStep = availableSteps[availableSteps.length - 1];
+      const currentInteractionStep = deepCopy(
+        availableSteps[availableSteps.length - 1]
+      );
+
       currentInteractionStep.question.filteredAnswerOptions =
         currentInteractionStep.question.answerOptions;
     }
diff --git a/src/components/utils.jsx b/src/components/utils.jsx
index 07df6bc3b..4f3037236 100644
--- a/src/components/utils.jsx
+++ b/src/components/utils.jsx
@@ -27,3 +27,16 @@ export function getButtonProps(props) {
   ];
   return pick(props, validProps);
 }
+
+// Create a deep copy of an object so nested properties are also mutable
+export function deepCopy(obj) {
+  if (Array.isArray(obj)) {
+    return obj.map(item => deepCopy(item));
+  } else if (typeof obj === "object" && obj !== null) {
+    return Object.fromEntries(
+      Object.entries(obj).map(([key, value]) => [key, deepCopy(value)])
+    );
+  } else {
+    return obj;
+  }
+}

From bcca02b5bd7b5b46ff5bcaedd975fe86cc520c4f Mon Sep 17 00:00:00 2001
From: "Maureen E. Zitouni" <maureen.zitouni@gmail.com>
Date: Fri, 29 Mar 2024 03:23:37 -0400
Subject: [PATCH 04/12] Define how fields should be merged (Apollo
 InMemoryCache error)

---
 src/network/apollo-client-singleton.js | 23 ++++++++++++++++++++++-
 1 file changed, 22 insertions(+), 1 deletion(-)

diff --git a/src/network/apollo-client-singleton.js b/src/network/apollo-client-singleton.js
index 59491dbe7..a86af1405 100644
--- a/src/network/apollo-client-singleton.js
+++ b/src/network/apollo-client-singleton.js
@@ -60,11 +60,32 @@ const cache = new InMemoryCache({
     }
     return null;
   },
-  // FUTURE: Apollo Client 3.0 allows this much more easily:
   typePolicies: {
     ContactTag: {
       // key is just the tag id and the value is contact-specific
       keyFields: false
+    },
+    // Define custom merge functions for multiple fields
+    // https://go.apollo.dev/c/merging-non-normalized-objects
+    // https://www.apollographql.com/docs/react/caching/cache-field-behavior/#merging-arrays
+    Query: {
+      fields: {
+        organization: {
+          merge: true
+        }
+      }
+    },
+    Campaign: {
+      fields: {
+        ingestMethod: {
+          merge: true
+        },
+        pendingJobs: {
+          merge(existing = [], incoming) {
+            return incoming;
+          }
+        }
+      }
     }
   }
 });

From 604a2a4c7f0c127440cc5ee056ac12d3b439715f Mon Sep 17 00:00:00 2001
From: "Maureen E. Zitouni" <maureen.zitouni@gmail.com>
Date: Fri, 29 Mar 2024 05:28:24 -0400
Subject: [PATCH 05/12] Parse userId when fetching and updating account

---
 src/containers/UserEdit.jsx | 42 +++++++++++++++++++++----------------
 1 file changed, 24 insertions(+), 18 deletions(-)

diff --git a/src/containers/UserEdit.jsx b/src/containers/UserEdit.jsx
index 2f52d74d5..ec9fcb916 100644
--- a/src/containers/UserEdit.jsx
+++ b/src/containers/UserEdit.jsx
@@ -62,23 +62,29 @@ const styles = StyleSheet.create({
   }
 });
 
-const fetchUser = async (organizationId, userId) =>
-  apolloClient.query({
-    query: gql`
-      query getEditedUser($organizationId: String!, $userId: Int!) {
-        user(organizationId: $organizationId, userId: $userId) {
-          id
-          firstName
-          email
-          lastName
-          alias
-          cell
-          extra
+const fetchUser = async (organizationId, userId) => {
+  try {
+    const response = await apolloClient.query({
+      query: gql`
+        query getEditedUser($organizationId: String!, $userId: Int!) {
+          user(organizationId: $organizationId, userId: $userId) {
+            id
+            firstName
+            email
+            lastName
+            alias
+            cell
+            extra
+          }
         }
-      }
-    `,
-    variables: { organizationId, userId }
-  });
+      `,
+      variables: { organizationId, userId }
+    });
+    return response;
+  } catch (error) {
+    console.error("Error fetching user:", error);
+  }
+};
 
 const fetchOrg = async organizationId =>
   apolloClient.query({
@@ -130,7 +136,7 @@ export class UserEditBase extends React.Component {
     if (!this.props.authType && this.props.userId) {
       const response = await fetchUser(
         this.props.organizationId,
-        this.props.userId
+        parseInt(this.props.userId)
       );
       this.setState({
         editedUser: response.data
@@ -483,7 +489,7 @@ const mutations = {
   editUser: ownProps => userData => ({
     mutation: editUserMutation,
     variables: {
-      userId: ownProps.userId,
+      userId: parseInt(ownProps.userId),
       organizationId: ownProps.organizationId,
       userData
     }

From 5831b033a771f6b4af813cfe3b463c362b557442 Mon Sep 17 00:00:00 2001
From: "Maureen E. Zitouni" <maureen.zitouni@gmail.com>
Date: Fri, 29 Mar 2024 05:28:46 -0400
Subject: [PATCH 06/12] Linter fix

---
 src/containers/UserEdit.jsx | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/containers/UserEdit.jsx b/src/containers/UserEdit.jsx
index ec9fcb916..d62535a47 100644
--- a/src/containers/UserEdit.jsx
+++ b/src/containers/UserEdit.jsx
@@ -513,9 +513,7 @@ const mutations = {
 };
 
 const UserEdit = withMuiTheme(
-  withRouter(
-    loadData({ queries, mutations})(UserEditBase)
-  )
+  withRouter(loadData({ queries, mutations })(UserEditBase))
 );
 
 export default UserEdit;

From d7759d0b348c93326a24e695980d52ff848d390f Mon Sep 17 00:00:00 2001
From: "Maureen E. Zitouni" <maureen.zitouni@gmail.com>
Date: Fri, 29 Mar 2024 08:12:00 -0400
Subject: [PATCH 07/12] Parse user id when getting todos data

---
 src/containers/TexterTodoList.jsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/containers/TexterTodoList.jsx b/src/containers/TexterTodoList.jsx
index 8e97c9c47..8388c3c7d 100644
--- a/src/containers/TexterTodoList.jsx
+++ b/src/containers/TexterTodoList.jsx
@@ -267,7 +267,7 @@ const queries = {
     query: dataQuery,
     options: ownProps => ({
       variables: {
-        userId: ownProps.params.userId || null,
+        userId: parseInt(ownProps.params.userId) || null,
         organizationId: ownProps.params.organizationId,
         todosOrg:
           ownProps.location.query["org"] == "all" ||
@@ -302,7 +302,7 @@ const queries = {
         );
       return {
         variables: {
-          userId: ownProps.params.userId || null,
+          userId: parseInt(ownProps.params.userId) || null,
           organizationId: ownProps.params.organizationId
         },
         fetchPolicy: "network-only",

From c5da67b18f0897341250e89a556f6379ca265e57 Mon Sep 17 00:00:00 2001
From: "Maureen E. Zitouni" <maureen.zitouni@gmail.com>
Date: Fri, 29 Mar 2024 08:15:23 -0400
Subject: [PATCH 08/12] Clean up

---
 src/containers/AdminIncomingMessageList.jsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/containers/AdminIncomingMessageList.jsx b/src/containers/AdminIncomingMessageList.jsx
index 037d8f165..0f3c7a003 100644
--- a/src/containers/AdminIncomingMessageList.jsx
+++ b/src/containers/AdminIncomingMessageList.jsx
@@ -24,7 +24,7 @@ export class AdminIncomingMessageList extends Component {
   constructor(props) {
     super(props);
 
-    const query = props.location.query;
+    // const query = props.location.query;
     const filters = getConversationFiltersFromQuery(
       props.location.query,
       props.organization.organization.tags

From 0de875f5a2e80ff670a15428ed5bd3862f6246fd Mon Sep 17 00:00:00 2001
From: "Maureen E. Zitouni" <maureen.zitouni@gmail.com>
Date: Fri, 29 Mar 2024 08:19:33 -0400
Subject: [PATCH 09/12] Add try/catch

---
 src/containers/PaginatedUsersRetriever.jsx | 27 ++++++++++++++--------
 1 file changed, 18 insertions(+), 9 deletions(-)

diff --git a/src/containers/PaginatedUsersRetriever.jsx b/src/containers/PaginatedUsersRetriever.jsx
index c9b260ba6..bfb5b567a 100644
--- a/src/containers/PaginatedUsersRetriever.jsx
+++ b/src/containers/PaginatedUsersRetriever.jsx
@@ -91,16 +91,25 @@ export class PaginatedUsersRetriever extends Component {
     let offset = 0;
     let total = undefined;
     let users = [];
+    let results;
     do {
-      const results = await fetchPeople(
-        offset,
-        pageSize,
-        organizationId,
-        campaignsFilter,
-        sortBy || "FIRST_NAME",
-        roleFilter
-      );
-      const { pageInfo, users: newUsers } = results.data.people;
+      try {
+        results = await fetchPeople(
+          offset,
+          pageSize,
+          organizationId,
+          campaignsFilter,
+          sortBy || "FIRST_NAME",
+          roleFilter
+        );
+      } catch (error) {
+        console.error("Error fetching people:", error);
+      }
+
+      const { pageInfo, users: newUsers } = results?.data?.people || {
+        pageInfo: {},
+        users: []
+      };
       users = users.concat(newUsers);
       offset += pageSize;
       total = pageInfo.total;

From 2892c96bf5dd491e4e1157835659b9edba9262a5 Mon Sep 17 00:00:00 2001
From: "Maureen E. Zitouni" <maureen.zitouni@gmail.com>
Date: Fri, 29 Mar 2024 08:45:22 -0400
Subject: [PATCH 10/12] Parse buying phone number limit

---
 src/containers/AdminPhoneNumberInventory.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/containers/AdminPhoneNumberInventory.js b/src/containers/AdminPhoneNumberInventory.js
index 9ba03ccb1..d23bb6cd4 100644
--- a/src/containers/AdminPhoneNumberInventory.js
+++ b/src/containers/AdminPhoneNumberInventory.js
@@ -492,7 +492,7 @@ const mutations = {
     variables: {
       organizationId: ownProps.params.organizationId,
       areaCode,
-      limit
+      limit: parseInt(limit)
     },
     refetchQueries: () => ["getOrganizationData"]
   }),

From 8608d07f3b1665b5d867e04de2bb1135c6924037 Mon Sep 17 00:00:00 2001
From: "Maureen E. Zitouni" <maureen.zitouni@gmail.com>
Date: Fri, 29 Mar 2024 08:55:52 -0400
Subject: [PATCH 11/12] Target limit table cell more precisely for integration
 tests

---
 __test__/cypress/integration/phone-inventory.test.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/__test__/cypress/integration/phone-inventory.test.js b/__test__/cypress/integration/phone-inventory.test.js
index 243e6247f..aa176c589 100644
--- a/__test__/cypress/integration/phone-inventory.test.js
+++ b/__test__/cypress/integration/phone-inventory.test.js
@@ -34,7 +34,9 @@ describe("Phone number management screen in the Admin interface", () => {
     // Waits until job run completes
     cy.waitUntil(
       () =>
-        cy.get(`tr:contains(${testAreaCode}) td:nth-child(4)`).contains("1"),
+        cy
+          .get(`tr:contains(${testAreaCode}) td:nth-child(4) div`)
+          .contains("1"),
       { timeout: 1000 }
     );
   });

From cf9e5a9b5a3375e0de48ea654b527439a230c23f Mon Sep 17 00:00:00 2001
From: "Maureen E. Zitouni" <maureen.zitouni@gmail.com>
Date: Fri, 29 Mar 2024 09:02:56 -0400
Subject: [PATCH 12/12] Clean up

---
 src/containers/AdminIncomingMessageList.jsx | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/containers/AdminIncomingMessageList.jsx b/src/containers/AdminIncomingMessageList.jsx
index 0f3c7a003..46f892c36 100644
--- a/src/containers/AdminIncomingMessageList.jsx
+++ b/src/containers/AdminIncomingMessageList.jsx
@@ -24,7 +24,6 @@ export class AdminIncomingMessageList extends Component {
   constructor(props) {
     super(props);
 
-    // const query = props.location.query;
     const filters = getConversationFiltersFromQuery(
       props.location.query,
       props.organization.organization.tags