fix: B2BTEAM-3592 Quick Order More for Less promo quantities - #187
Conversation
…k Order Checkout simulation quantity can differ from user input when More for Less promotions apply. Stop treating that field as fulfillable stock unless availability indicates a real stock shortfall, and preserve requested quantities for validation and review display. Co-authored-by: Wender <wender@users.noreply.github.com>
More for Less simulations return separate paid and gift lines for the same SKU and seller; concatenating them made getSkuSellerInfo use only the first line quantity. Sum quantities and merge priceTags when aggregating. Co-authored-by: Wender <wender@users.noreply.github.com>
|
Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖 Please select which version do you want to release:
And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.
|
|
Beep boop 🤖 I noticed you didn't make any changes at the
In order to keep track, I'll create an issue if you decide now is not a good time
|
Resolve CHANGELOG.md conflict: keep Unreleased Fixed entries and 3.16.9 release notes. Co-authored-by: Wender <wender@users.noreply.github.com>
nicholas-maestrello
left a comment
nicholas-maestrello
left a comment
There was a problem hiding this comment.
Approve no merge das linhas split do More for Less (paid + gift) por seller — isso cobre o Max=2 no 1101,3.
Deixei uma observação no heurístico de priceTags: não bloqueia o merge desta KI, mas vale um follow-up para não mascarar estoque parcial real.
Remove the broad priceTags heuristic from promotion-adjusted quantity detection so Validate reports partiallyAvailable when simulated qty is below the request (progressive discount, MFL + low stock). MFL paid+gift split lines remain handled via seller aggregation. Co-authored-by: Wender <wender@users.noreply.github.com>
|
Your PR has been merged! App is being published. 🚀 After the publishing process has been completed (check #vtex-io-releases) and doing A/B tests with the new version, you can deploy your release by running:
After that your app will be updated on all accounts. For more information on the deployment process check the docs. 📖 |
3 participants
Summary
Fixes Quick Order treating the first More for Less split line quantity as the max available units (e.g. Buy3Get1 → Max=2 on paste
1101,3).Jira: B2BTEAM-3592
Validation (b2bstoreqa)
1101,3→Max quantity is 2(FAIL)qamoreforlesswith this branch linked (338cdb1): Valid qty 3, Add to Cart OK, cart 2 paid + 1 Free (PASS)Test plan
b2bstoreqa, promo Buy3Get1 active, SKU 11 / RefId 11011101,3→ Validate without Max=2