update feature/manual-bonus-products-v3 with latest changes in develop branch#3286
Merged
sf-shikhar-prasoon merged 24 commits intofeature/manual-bonus-products-v3from Sep 10, 2025
Merged
Conversation
* bump versions and update changelogs * Add breaking label to multi-ship changelog entry * snyc to -preview.2
* bump versions and update changelogs * Add breaking label to multi-ship changelog entry * snyc to -preview.2 * @W-19442561 - Fix Incorrectly Disabled Continue to Shipping Method Button for Multiship (#3199) * @W-19443375,@W-19350149 MULTISHIP fix address function mismatch and options promotions (#3203) * @W-19451314 - Add ShipmentId to Create Shipment Body (#3211) * @W-19349040 MULTISHIP fix form default delivery option selection (#3217) * @ W-19450629 feat: remove quantity selector from shipping methods preview cart in multi-ship (#3216) * remove quantity selector from shipping methods preview cart in multi ship * update change log --------- Signed-off-by: sf-henry-semaganda <henry.semaganda@salesforce.com> * Fix non-extensible template to use correct passwordless login callback uri (#3222) * Check if monorepo is a dev version before publishing to npm (#3210) * Check if monorepo is a dev version before publishing to npm * Check env.IS_DEV_VERSION * test step * test * Apply suggestion * More test * test * test * test * use if block * test * test * use quiet mode * test * test * test * test * test * test * test * Try this regex that's perhaps more compatible * Re-enable other steps and clean up verification step --------- Co-authored-by: Vincent Marta <vmarta@salesforce.com> * Merge pull request #3204 from SalesforceCommerceCloud/bugfix/pin-testing-library-version @W-19443536 - [Bugfix] Fix unit test failures in generated projects * Fix: Don't pass `code_challenge` if its undefined in `getPasswordResetToken` (#3228) * add fix * make code challenge optional * lint --------- Co-authored-by: vcua-mobify <47404250+vcua-mobify@users.noreply.github.com> * @W-19364221 - Handle Removal of Last Address for Registered (#3231) * Gracefully handle when some config.app.* properties are missing (@W-19453183@) (#3230) * Don't assume that config.app.* properties exist * Don't assume that config.app.* properties exist * Add todos * Use the default config for Commerce Agent when necessary * Debugging * Move utils file to somewhere more upgradable and extensible * No longer need these .hbs files * Sync the corresponding .hbs files * Fix import path * Fix require path so it can work inside monorepo * Refactor to make it easier to get commerce agent config * Fix module error Turns out that I can't mix both esm and commonjs ways. * Sync the eslint comments * Remove commented out code * Update CHANGELOG.md --------- Signed-off-by: Vincent Marta <vmarta@salesforce.com> Co-authored-by: vcua-mobify <47404250+vcua-mobify@users.noreply.github.com> * @W-19364130 feat: fix shipping costs summary mode pricing (#3240) * fix shipping costs sumary mode * update change log * add clear annotation * add test coverage * update basket too * update * bump versions to -preview.3 * Merge pull request #3214 from SalesforceCommerceCloud/restore-per-pr-e2e Restore e2e-PR workflow * Fix e2e-pr after incorrect merge resolution * preview.3 not preview-3 --------- Signed-off-by: sf-henry-semaganda <henry.semaganda@salesforce.com> Signed-off-by: Vincent Marta <vmarta@salesforce.com> Co-authored-by: shauryemahajanSF <143645277+shauryemahajanSF@users.noreply.github.com> Co-authored-by: patricksullivansf <100441292+patricksullivansf@users.noreply.github.com> Co-authored-by: sf-henry-semaganda <henry.semaganda@salesforce.com> Co-authored-by: Vincent Marta <vmarta@salesforce.com> Co-authored-by: Jainam Sheth <99490559+shethj@users.noreply.github.com> Co-authored-by: Joel Uong <88680517+joeluong-sfcc@users.noreply.github.com>
It's not as helpful because it can confuse other people and cause false alarm. One use of this function is to parse an optional environment variable. So if it is not set, it'll constantly prints out the warning message.
[release 3.12.0] Bump up to final/stable versions
…-to-develop Merge v3.12.0 release back into `develop` branch
…545821/updateToolDescription @W-19545821 - Refactoring parameter names
…k/W-19364102/CartPickupFigmaMatch @W-19364102 - Cart pick up section changes in StoreDisplay component to match figma
Collaborator
⛔ Snyk checks have failed. 1 issues have been found so far.
⛔ security/snyk check is complete. 1 issues have been found. (View Details) ✅ license/snyk check is complete. No issues have been found. (View Details) 💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
sf-xingquan-jin
approved these changes
Sep 10, 2025
52ce135
into
feature/manual-bonus-products-v3
53 of 83 checks passed
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
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