fix(core): show success message when a subscribed user resubscribes

Author: jorgemoya

.changeset/upset-bushes-hang.md

@@ -0,0 +1,5 @@
+---
+"@bigcommerce/catalyst-core": minor
+---
+
+Show success message when a subscribed is already subscribed to newsletter.

core/components/subscribe/_actions/subscribe.ts

@@ -57,18 +57,22 @@ export const subscribe = async (
 
     const errors = response.data.newsletter.subscribe.errors;
 
-    if (!errors.length) {
-      return { lastResult: submission.reply({ resetForm: true }), successMessage: t('success') };
+    // If there are no errors or the error is that the email already exists, we want to reset the form and show the success message
+    if (
+      !errors.length ||
+      errors.some(({ __typename }) => __typename === 'CreateSubscriberAlreadyExistsError')
+    ) {
+      return {
+        lastResult: submission.reply(),
+        successMessage: t('subscribedToNewsletter'),
+      };
     }
 
     if (errors.length > 0) {
       return {
         lastResult: submission.reply({
           formErrors: errors.map(({ __typename }) => {
             switch (__typename) {
-              case 'CreateSubscriberAlreadyExistsError':
-                return t('Errors.subcriberAlreadyExists');
-
               case 'CreateSubscriberEmailInvalidError':
                 return t('Errors.invalidEmail');
 

core/messages/en.json

@@ -504,9 +504,8 @@
       "title": "Sign up for our newsletter",
       "placeholder": "Enter your email",
       "description": "Stay up to date with the latest news and offers from our store.",
-      "success": "You have been subscribed to our newsletter.",
+      "subscribedToNewsletter": "You have been subscribed to our newsletter.",
       "Errors": {
-        "subcriberAlreadyExists": "You are already subscribed to our newsletter.",
         "invalidEmail": "Please enter a valid email address.",
         "somethingWentWrong": "Something went wrong. Please try again later."
       }

core/tests/ui/e2e/subscribe.spec.ts

@@ -24,7 +24,7 @@ test(
     await submitButton.click();
     await page.waitForLoadState('networkidle');
 
-    await expect(page.getByText(t('success'))).toBeVisible();
+    await expect(page.getByText(t('subscribedToNewsletter'))).toBeVisible();
 
     subscribe.trackSubscription(email);
   },
@@ -50,14 +50,14 @@ test('Shows error when user tries to subscribe again with the same email', async
   await submitButton.click();
   await page.waitForLoadState('networkidle');
 
-  await expect(page.getByText(t('success'))).toBeVisible();
+  await expect(page.getByText(t('subscribedToNewsletter'))).toBeVisible();
 
   // Try to subscribe again with the same email
   await emailInput.fill(email);
   await submitButton.click();
   await page.waitForLoadState('networkidle');
 
-  await expect(page.getByText(t('Errors.subcriberAlreadyExists'))).toBeVisible();
+  await expect(page.getByText(t('subscribedToNewsletter'))).toBeVisible();
 
   // Track that we attempted to subscribe this email
   subscribe.trackSubscription(email);