From 0541afceb8855511160fcc74944f3b42b4862f93 Mon Sep 17 00:00:00 2001
From: Dave Richer <dave@imexsystems.ca>
Date: Wed, 21 May 2025 15:17:11 -0400
Subject: [PATCH] feature/IO-3182-Phone-Number-Consent - Checkpoint

---
 .../chat-conversation-list.component.jsx      |  5 +----
 .../phone-number-consent.component.jsx        |  9 +++++----
 .../graphql/phone-number-opt-out.queries.js   |  6 +++---
 client/src/translations/en_us/common.json     |  2 +-
 hasura/metadata/tables.yaml                   | 19 +++++++++++++++++++
 5 files changed, 29 insertions(+), 12 deletions(-)

diff --git a/client/src/components/chat-conversation-list/chat-conversation-list.component.jsx b/client/src/components/chat-conversation-list/chat-conversation-list.component.jsx
index 70bfc5b5d..8bf185756 100644
--- a/client/src/components/chat-conversation-list/chat-conversation-list.component.jsx
+++ b/client/src/components/chat-conversation-list/chat-conversation-list.component.jsx
@@ -62,10 +62,7 @@ function ChatConversationListComponent({ conversationList, selectedConversation,
   const renderConversation = (index, t) => {
     const item = sortedConversationList[index];
     const normalizedPhone = phone(item.phone_num, "CA").phoneNumber.replace(/^\+1/, "");
-    // Check if the phone number exists in the consentMap
     const hasOptOutEntry = optOutMap.has(normalizedPhone);
-    // Only consider it non-consented if it exists and consent_status is false
-    const isOptedOut = hasOptOutEntry ? optOutMap.get(normalizedPhone) : true;
 
     const cardContentRight = <TimeAgoFormatter>{item.updated_at}</TimeAgoFormatter>;
     const cardContentLeft =
@@ -91,7 +88,7 @@ function ChatConversationListComponent({ conversationList, selectedConversation,
     const cardExtra = (
       <>
         <Badge count={item.messages_aggregate.aggregate.count} />
-        {hasOptOutEntry && !isOptedOut && <Tag color="red">{t("messaging.labels.no_consent")}</Tag>}
+        {hasOptOutEntry && <Tag color="red">{t("messaging.labels.no_consent")}</Tag>}
       </>
     );
 
diff --git a/client/src/components/phone-number-consent/phone-number-consent.component.jsx b/client/src/components/phone-number-consent/phone-number-consent.component.jsx
index fa0f810c0..020bb1750 100644
--- a/client/src/components/phone-number-consent/phone-number-consent.component.jsx
+++ b/client/src/components/phone-number-consent/phone-number-consent.component.jsx
@@ -39,9 +39,10 @@ function PhoneNumberConsentList({ bodyshop, currentUser }) {
       sorter: (a, b) => a.phone_number.localeCompare(b.phone_number)
     },
     {
-      title: t("consent.updated_at"),
-      dataIndex: "consent_updated_at",
-      render: (text) => <TimeAgoFormatter>{text}</TimeAgoFormatter>
+      title: t("consent.created_at"),
+      dataIndex: "created_at",
+      render: (text) => <TimeAgoFormatter>{text}</TimeAgoFormatter>,
+      sorter: (a, b) => new Date(a.created_at) - new Date(b.created_at)
     }
   ];
 
@@ -55,7 +56,7 @@ function PhoneNumberConsentList({ bodyshop, currentUser }) {
 
       <Table
         columns={columns}
-        dataSource={data?.phone_number_consent}
+        dataSource={data?.phone_number_opt_out}
         loading={loading}
         rowKey="id"
         style={{ marginTop: 16 }}
diff --git a/client/src/graphql/phone-number-opt-out.queries.js b/client/src/graphql/phone-number-opt-out.queries.js
index d77e0aa25..861bdf64d 100644
--- a/client/src/graphql/phone-number-opt-out.queries.js
+++ b/client/src/graphql/phone-number-opt-out.queries.js
@@ -2,7 +2,7 @@ import { gql } from "@apollo/client";
 
 export const GET_PHONE_NUMBER_OPT_OUT = gql`
   query GET_PHONE_NUMBER_OPT_OUT($bodyshopid: uuid!, $phone_number: String!) {
-    phone_number_consent(where: { bodyshopid: { _eq: $bodyshopid }, phone_number: { _eq: $phone_number } }) {
+    phone_number_opt_out(where: { bodyshopid: { _eq: $bodyshopid }, phone_number: { _eq: $phone_number } }) {
       id
       bodyshopid
       phone_number
@@ -14,9 +14,9 @@ export const GET_PHONE_NUMBER_OPT_OUT = gql`
 
 export const GET_PHONE_NUMBER_OPT_OUTS = gql`
   query GET_PHONE_NUMBER_OPT_OUTS($bodyshopid: uuid!, $search: String) {
-    phone_number_consent(
+    phone_number_opt_out(
       where: { bodyshopid: { _eq: $bodyshopid }, phone_number: { _ilike: $search } }
-      order_by: [{ phone_number: asc }, { consent_updated_at: desc }]
+      order_by: [{ phone_number: asc }, { updated_at: desc }]
     ) {
       id
       bodyshopid
diff --git a/client/src/translations/en_us/common.json b/client/src/translations/en_us/common.json
index 67ddcb72d..e456f6b22 100644
--- a/client/src/translations/en_us/common.json
+++ b/client/src/translations/en_us/common.json
@@ -3869,7 +3869,7 @@
     "consent": {
       "phone_number": "Phone Number",
       "status": "Consent Status",
-      "updated_at": "Last Updated"
+      "created_at": "Created At"
     },
     "settings": {
       "title": "Phone Number Opt-Out List"
diff --git a/hasura/metadata/tables.yaml b/hasura/metadata/tables.yaml
index 025166032..c43b4e1dd 100644
--- a/hasura/metadata/tables.yaml
+++ b/hasura/metadata/tables.yaml
@@ -5868,6 +5868,25 @@
     - name: bodyshop
       using:
         foreign_key_constraint_on: bodyshopid
+  select_permissions:
+    - role: user
+      permission:
+        columns:
+          - phone_number
+          - created_at
+          - updated_at
+          - bodyshopid
+          - id
+        filter:
+          bodyshop:
+            associations:
+              _and:
+                - user:
+                    authid:
+                      _eq: X-Hasura-User-Id
+                - active:
+                    _eq: true
+      comment: ""
 - table:
     name: phonebook
     schema: public