@W-18999015 footer component updated for shopper consents email marketing subscription hooks#3471
Merged
mitesh-patel-crm merged 4 commits intofeature/develop-shopper-consent-marketing-preferencesfrom Nov 18, 2025
Conversation
Queries shopper-consents SCAPI for tags=email_capture, and then updates all subscriptions to opt_in given the input email address as the contactPointValue.
Collaborator
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
mitesh-patel-crm
requested review from
knhage and
kzheng-sfdc
and removed request for
a team
kzheng-sfdc
reviewed
Nov 17, 2025
| const tagList = tags.join(', ') | ||
| console.error( | ||
| `[useEmailSubscription] No subscriptions found for tag(s) "${tagList}" and channel "email". ` + | ||
| `Please configure subscriptions in Business Manager with one of these tags: ${tagList}.` |
Contributor
There was a problem hiding this comment.
This error seems to be more suitable for server side logging than for browser/shopper.
Collaborator
Author
There was a problem hiding this comment.
yeah, it shouldn't say anything about Business Manager!! Let me fix that.
kzheng-sfdc
approved these changes
Nov 17, 2025
| * | ||
| * Subscriptions are fetched on-demand when the user clicks submit, not on component mount. | ||
| * | ||
| * This allows marketers to configure subscriptions in Business Manager without code changes. |
Collaborator
Author
There was a problem hiding this comment.
@kzheng-sfdc is it ok to reference Business Manager here? I see "Business Manager" referenced in a README file for pwa-kit. I could also be more vague here instead.
mitesh-patel-crm
merged commit 1ec047a
into
feature/develop-shopper-consent-marketing-preferences
13 of 16 checks passed
mitesh-patel-crm
deleted the
mitesh-patel-crm/w-18999015-shopper-consents-footer-only
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
W-18999015 Footer Component hooks for Marketing Consent.
Queries shopper-consents SCAPI for tags=email_capture, and then updates all subscriptions to opt_in given the input email address as the contactPointValue.
Description
Types of Changes
Changes
How to Test-Drive This PR
Checklists
General
Accessibility Compliance
You must check off all items in one of the follow two lists:
or...
Localization