[Extensibility ⚙️] Update feature branch from develop @@W-18023320@@ WIP#2325
Merged
bendvc merged 85 commits intofeature/extensibility-v2from Mar 27, 2025
Merged
Conversation
Note: Unsure about whether mockedRegisteredCustomerWithNoNumber should have been re-added
Re-worked the data cloud tests, still a work in progress.
Skip header tests Skip datacloud tests
Remove ci bundle analyzer action
kevinxh
approved these changes
Mar 26, 2025
Contributor
kevinxh
left a comment
There was a problem hiding this comment.
LGTM. Smoked tested both locally and on MRT. Storefront looks good and I'm able to complete shopper happy path.
Since this refactor is too big, I did not go over every diff.
adamraya
reviewed
Mar 26, 2025
Contributor
There was a problem hiding this comment.
I see this error starting a generated project:
ModuleNotFoundError: Module not found: Error: Can't resolve '@salesforce/cc-datacloud-typescript' in '/Users/arayanavarro/git/v2-pwa-kit/test/node_modules/@salesforce/extension-chakra-storefront/src/hooks'
at /Users/arayanavarro/git/v2-pwa-kit/test/node_modules/webpack/lib/Compilation.js:2123:28
at /Users/arayanavarro/git/v2-pwa-kit/test/node_modules/webpack/lib/NormalModuleFactory.js:923:13
at eval (eval at create (/Users/arayanavarro/git/v2-pwa-kit/test/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:10:1)
at /Users/arayanavarro/git/v2-pwa-kit/test/node_modules/webpack/lib/NormalModuleFactory.js:339:22
at eval (eval at create (/Users/arayanavarro/git/v2-pwa-kit/test/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:9:1)
Contributor
Author
Fixed. I updated the storefront extension to define the additional dependencies as peer deps. E.g. cc-datacloud and jose |
adamraya
reviewed
Mar 26, 2025
Contributor
There was a problem hiding this comment.
I see a few Content Security Policy errors in the browser console.
E.g.
vendor.js:281 Refused to connect to 'https://mmyw8zrxhfsg09lfmzrd1zjqmg.pc-rnd.c360a.salesforce.com/web/events/fb81edab-24c6-4b40-8684-b67334dfdf32' because it violates the following Content Security Policy directive: "connect-src api.cquotient.com 'self' https://runtime.commercecloud.com".
Did we missed merging the updated CSP?
adamraya
approved these changes
Mar 26, 2025
alexvuong
pushed a commit
that referenced
this pull request
Jul 16, 2025
…20_update-storefront [Extensibility ⚙️] Update feature branch from `develop` @@W-18023320@@ WIP
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
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.
Description
What is this PR
A lot of progress for features has been made in the retail react application template in develop. Unfortunately all these changes were not made in the context of Application Extensibility. That means that since we had created the storefront extension a long time ago it diverged from the template.
Some notable changes that we missed are DNT, Social Login and a hand full of other changes like accessibility work.
In this PR we take the content of develop and re-create the
extension-chakra-storefrontto insure that all the changes are carried over, and thus, going forward we will remove the template project from the extensibility branch, and any change that is made todevelop template-retail-react-appthat change will have to be made to the extension as well.Methodology
To start this process I follow the below basic steps:
feature/extensibility-v2and resolved any merge conflicts, this mainly included things like change logs and version numbers of mono repo packages.extension-chakra-storefronttoextension-chakra-storefront-old. (This allows to me keep any changes that we made to the project to make things with with extensibility, including changes to tests to make them work.) NOTE: I usedgit mvto do this to keep all commit history.template-retail-react-appto beextension-chakra-storefront. I again used git mv to preserve the git history.getConfigwithuseExtensionConfigetcHow to test?
Notes
Types of Changes
Changes
How to Test-Drive This PR