From 0dfcea59a092b1d136bf3f26b5410f1c1b94313e Mon Sep 17 00:00:00 2001
From: sjwmoveon <sophie.waldman@moveon.org>
Date: Fri, 3 May 2024 18:12:37 -0400
Subject: [PATCH] Use 'joinToken' in place of 'organizationUuid' since we are
 actually using the joinToken value Also remove an unnecessary import and fix
 a comment

---
 __test__/server/api/campaign/campaign.test.js    |  8 ++++----
 src/api/schema.js                                |  2 +-
 src/components/CampaignDynamicAssignmentForm.jsx |  2 +-
 src/components/OrganizationReassignLink.jsx      |  6 +++---
 src/containers/AssignReplies.jsx                 | 10 +++++-----
 src/routes.jsx                                   |  2 +-
 src/server/api/schema.js                         |  6 +++---
 7 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/__test__/server/api/campaign/campaign.test.js b/__test__/server/api/campaign/campaign.test.js
index da3b0eac3..086a32c72 100644
--- a/__test__/server/api/campaign/campaign.test.js
+++ b/__test__/server/api/campaign/campaign.test.js
@@ -846,13 +846,11 @@ describe("Reassignments", () => {
     await runGql(
       dynamicReassignMutation,
       {
-        organizationUuid: testCampaign.joinToken,
+        joinToken: testCampaign.joinToken,
         campaignId: testCampaign.id,
       },
       testTexterUser2
     );
-     // TEXTER 1 (60 needsMessage, 4 messaged)
-    // TEXTER 2 (25 needsMessage, 2 needsResponse, 3 convo, 1 messaged)
     texterCampaignDataResults = await runGql(
       TexterTodoQuery,
       {
@@ -897,7 +895,7 @@ describe("Reassignments", () => {
     await runGql(
       dynamicReassignMutation,
       {
-        organizationUuid: testCampaign.joinToken,
+        joinToken: testCampaign.joinToken,
         campaignId: testCampaign.id,
       },
       testTexterUser2
@@ -930,6 +928,8 @@ describe("Reassignments", () => {
       },
       testTexterUser2
     );
+    // TEXTER 1 (60 needsMessage, 4 messaged)
+    // TEXTER 2 (25 needsMessage, 2 needsResponse, 3 convo, 1 messaged)
     expect(texterCampaignDataResults.data.assignment.contacts.length).toEqual(
       0
     );
diff --git a/src/api/schema.js b/src/api/schema.js
index a87157cbb..fee8d89a5 100644
--- a/src/api/schema.js
+++ b/src/api/schema.js
@@ -399,7 +399,7 @@ const rootSchema = gql`
       newTexterUserId: String!
     ): [CampaignIdAssignmentId]
     dynamicReassign(
-      organizationUuid: String!
+      joinToken: String!
       campaignId: String!
     ): String
     importCampaignScript(campaignId: String!, url: String!): Int
diff --git a/src/components/CampaignDynamicAssignmentForm.jsx b/src/components/CampaignDynamicAssignmentForm.jsx
index e12f8c56a..ae25dbab5 100644
--- a/src/components/CampaignDynamicAssignmentForm.jsx
+++ b/src/components/CampaignDynamicAssignmentForm.jsx
@@ -156,7 +156,7 @@ class CampaignDynamicAssignmentForm extends React.Component {
                 <li>
                   {joinToken ? (
                     <OrganizationReassignLink
-                      organizationUuid={joinToken}
+                      joinToken={joinToken}
                       campaignId={campaignId}
                     />
                   ) : (
diff --git a/src/components/OrganizationReassignLink.jsx b/src/components/OrganizationReassignLink.jsx
index 1a3fd388b..bdcc1a315 100644
--- a/src/components/OrganizationReassignLink.jsx
+++ b/src/components/OrganizationReassignLink.jsx
@@ -2,20 +2,20 @@ import PropTypes from "prop-types";
 import React from "react";
 import DisplayLink from "./DisplayLink";
 
-const OrganizationReassignLink = ({ organizationUuid, campaignId }) => {
+const OrganizationReassignLink = ({ joinToken, campaignId }) => {
   let baseUrl = "http://base";
   if (typeof window !== "undefined") {
     baseUrl = window.location.origin;
   }
 
-  const replyUrl = `${baseUrl}/${organizationUuid}/replies/${campaignId}`;
+  const replyUrl = `${baseUrl}/${joinToken}/replies/${campaignId}`;
   const textContent = `Send your texting volunteers this link! Once they sign up, they\'ll be automatically assigned replies for this campaign.`;
 
   return <DisplayLink url={replyUrl} textContent={textContent} />;
 };
 
 OrganizationReassignLink.propTypes = {
-  organizationUuid: PropTypes.string,
+  joinToken: PropTypes.string,
   campaignId: PropTypes.string
 };
 
diff --git a/src/containers/AssignReplies.jsx b/src/containers/AssignReplies.jsx
index aca51dad8..3e50c5418 100644
--- a/src/containers/AssignReplies.jsx
+++ b/src/containers/AssignReplies.jsx
@@ -22,7 +22,7 @@ class AssignReplies extends React.Component {
     try {
       
       const organizationId = (await this.props.mutations.dynamicReassign(
-        this.props.params.organizationUuid,
+        this.props.params.joinToken,
         this.props.params.campaignId
       )).data.dynamicReassign;
       console.log("ID:", organizationId);
@@ -62,11 +62,11 @@ AssignReplies.propTypes = {
 
 export const dynamicReassignMutation = gql`
   mutation dynamicReassign(
-    $organizationUuid: String!
+    $joinToken: String!
     $campaignId: String!
   ) {
     dynamicReassign(
-      organizationUuid: $organizationUuid
+      joinToken: $joinToken
       campaignId: $campaignId
     )
   }
@@ -74,12 +74,12 @@ export const dynamicReassignMutation = gql`
 
 const mutations = {
   dynamicReassign: ownProps => (
-    organizationUuid,
+    joinToken,
     campaignId
   ) => ({
     mutation: dynamicReassignMutation,
     variables: {
-      organizationUuid,
+      joinToken,
       campaignId
     }
   })
diff --git a/src/routes.jsx b/src/routes.jsx
index 47685bfa1..7fce13cfc 100644
--- a/src/routes.jsx
+++ b/src/routes.jsx
@@ -277,7 +277,7 @@ export default function makeRoutes(requireAuth = () => {}) {
         onEnter={requireAuth}
       />
       <Route
-        path=":organizationUuid/replies/:campaignId"
+        path=":joinToken/replies/:campaignId"
         component={AssignReplies}
         onEnter={requireAuth}
       />
diff --git a/src/server/api/schema.js b/src/server/api/schema.js
index 14f7d3127..c0abaa43b 100644
--- a/src/server/api/schema.js
+++ b/src/server/api/schema.js
@@ -2,7 +2,7 @@ import GraphQLDate from "graphql-date";
 import GraphQLJSON from "graphql-type-json";
 import { GraphQLError } from "graphql/error";
 import isUrl from "is-url";
-import _, { orderBy } from "lodash";
+import _ from "lodash";
 import { gzip, makeTree, getHighestRole } from "../../lib";
 import { capitalizeWord, groupCannedResponses } from "./lib/utils";
 import httpRequest from "../lib/http-request";
@@ -1441,7 +1441,7 @@ const rootMutations = {
     dynamicReassign: async (
       _,
       {
-        organizationUuid,
+        joinToken,
         campaignId
       },
       { user }
@@ -1451,7 +1451,7 @@ const rootMutations = {
       .knex("campaign")
       .where({
         id: campaignId,
-        join_token: organizationUuid,
+        join_token: joinToken,
       })
       .first();
       const INVALID_REASSIGN = () => {