Skip to content

Conversation

@jordanarldt
Copy link
Contributor

@jordanarldt jordanarldt commented Dec 30, 2025

What/Why?

Hide the "Exclusive Offers" field from the Account Registration form.

Currently, the field is not fully implemented in GraphQL, so it may be misleading to display it on the storefront if it's not actually doing anything when registering a customer.

Once the Register Customer operation takes this field into account, we can display it again.

Testing

Tested locally

Migration

Update core/app/[locale]/(default)/(auth)/register/page.tsx and add the function:

// There is currently a GraphQL gap where the "Exclusive Offers" field isn't accounted for
// during customer registration, so the field should not be shown on the Catalyst storefront until it is hooked up.
function removeExlusiveOffersField(field: Field | Field[]): boolean {
  if (Array.isArray(field)) {
    // Exclusive offers field will always have ID '25', since it is made upon store creation and is also read-only.
    return !field.some((f) => f.id === '25');
  }

  return field.id !== '25';
}

Then, add the following code at the end of the const fields declaration:

    })
    .filter(exists)
    .filter(removeExlusiveOffersField); // <---

@jordanarldt jordanarldt requested a review from a team as a code owner December 30, 2025 20:24
@changeset-bot
Copy link

changeset-bot bot commented Dec 30, 2025

🦋 Changeset detected

Latest commit: 511768f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@bigcommerce/catalyst-core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Dec 30, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
catalyst Ready Ready Preview, Comment Dec 30, 2025 8:28pm
1 Skipped Deployment
Project Deployment Review Updated (UTC)
catalyst-uplift-vertex Ignored Ignored Dec 30, 2025 8:28pm

chanceaclark
chanceaclark previously approved these changes Dec 30, 2025
@chanceaclark chanceaclark dismissed their stale review December 30, 2025 20:55

Didn't mean to approve without my question answered

@jordanarldt jordanarldt added this pull request to the merge queue Dec 30, 2025
Merged via the queue into canary with commit 57a3527 Dec 30, 2025
8 checks passed
@jordanarldt jordanarldt deleted the CATALYST-1665 branch December 30, 2025 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants