-
Notifications
You must be signed in to change notification settings - Fork 386
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
feature/CXCDS-12370 - DM- 281: Downstream - commerce site propagation to ISS - SPA component #19385
base: develop
Are you sure you want to change the base?
Conversation
…into feature/CXCDS-12370
projects/core/src/features-config/feature-toggles/config/feature-toggles.ts
Outdated
Show resolved
Hide resolved
…into feature/CXCDS-12370
spartacus Run #45354
Run Properties:
|
Project |
spartacus
|
Run status |
Passed #45354
|
Run duration | 04m 06s |
Commit |
d3908feb55 ℹ️: Merge 7acee197237d37e7baef2c5d09e6c13351ea9a0f into e9a1c2efef8eddb2824016021eb0...
|
Committer | SebastianHamilton |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
3
|
Pending |
2
|
Skipped |
0
|
Passing |
125
|
@@ -381,6 +381,7 @@ if (environment.cpq) { | |||
enableConsecutiveCharactersPasswordRequirement: true, | |||
enablePasswordsCannotMatchInPasswordUpdateForm: true, | |||
allPageMetaResolversEnabledInCsr: true, | |||
cdsCasEnabled: false, |
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.
Ideally, all new toggles should be disabled by default for customers, but enabled in our example storefrontapp (for more, see our Coding Guidelines).
But perhaps you might have some strong arguments to not enable this particular toggle in our storefrontapp. Then I'm happy to learn your point of view.
No description provided.