-
Notifications
You must be signed in to change notification settings - Fork 147
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Extensibility ⚙️] Update feature branch from develop
@@W-18023320@@ WIP
#2325
[Extensibility ⚙️] Update feature branch from develop
@@W-18023320@@ WIP
#2325
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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)
Fixed. I updated the storefront extension to define the additional dependencies as peer deps. E.g. cc-datacloud and jose |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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?
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-storefront
to 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-app
that change will have to be made to the extension as well.Methodology
To start this process I follow the below basic steps:
feature/extensibility-v2
and resolved any merge conflicts, this mainly included things like change logs and version numbers of mono repo packages.extension-chakra-storefront
toextension-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 mv
to do this to keep all commit history.template-retail-react-app
to beextension-chakra-storefront
. I again used git mv to preserve the git history.getConfig
withuseExtensionConfig
etcHow to test?
Notes
Types of Changes
Changes
How to Test-Drive This PR