fix messaging 'test connection' workflow on initial save (#2751)

ethyca · Mar 6, 2023 · e5edd4f · e5edd4f
1 parent 6ff436e
commit e5edd4f
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 3 deletions.
diff --git a/clients/admin-ui/src/features/privacy-requests/configuration/MailgunEmailConfiguration.tsx b/clients/admin-ui/src/features/privacy-requests/configuration/MailgunEmailConfiguration.tsx
@@ -166,7 +166,9 @@ const MailgunEmailConfiguration = () => {
       ) : null}
       {configurationStep === "testConnection" ? (
         <TestMessagingProviderConnectionButton
-          messagingDetails={messagingDetails}
+          messagingDetails={
+            messagingDetails || { service_type: messagingProviders.mailgun }
+          }
         />
       ) : null}
     </Box>

diff --git a/clients/admin-ui/src/features/privacy-requests/configuration/TwilioEmailConfiguration.tsx b/clients/admin-ui/src/features/privacy-requests/configuration/TwilioEmailConfiguration.tsx
@@ -163,7 +163,11 @@ const TwilioEmailConfiguration = () => {
       ) : null}
       {configurationStep === "testConnection" ? (
         <TestMessagingProviderConnectionButton
-          messagingDetails={messagingDetails}
+          messagingDetails={
+            messagingDetails || {
+              service_type: messagingProviders.twilio_email,
+            }
+          }
         />
       ) : null}
     </Box>

diff --git a/clients/admin-ui/src/features/privacy-requests/configuration/TwilioSMS.tsx b/clients/admin-ui/src/features/privacy-requests/configuration/TwilioSMS.tsx
@@ -117,7 +117,9 @@ const TwilioSMSConfiguration = () => {
       </Stack>
       {configurationStep === "testConnection" ? (
         <TestMessagingProviderConnectionButton
-          messagingDetails={messagingDetails}
+          messagingDetails={
+            messagingDetails || { service_type: messagingProviders.twilio_text }
+          }
         />
       ) : null}
     </Box>