-
Notifications
You must be signed in to change notification settings - Fork 683
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
PWA-3071::Graphql Cart > Prices > Discounts array also including Gift… #4052
Conversation
|
run cypress |
Successfully started codebuild job for |
run lighthouse-desktop |
Successfully started codebuild job for |
run cypress |
Successfully started codebuild job for |
run lighthouse-desktop |
Successfully started codebuild job for |
run lighthouse-mobile |
Successfully started codebuild job for |
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.
Approving PR based on QA PASS on https://jira.corp.adobe.com/browse/PWA-3071
QA PASS on https://jira.corp.adobe.com/browse/PWA-3071 |
run cypress |
Successfully started codebuild job for |
just fyi checking for the label of something will fail the moment the language is not English and/or the term was translated by the back-end. |
… Cards in 2.4.6
Description
URL - https://pr-4030.pwa-venia.com/
Steps -
Issue - As per 2.4.5, Prices > Discounts array should not contain gift cards. But on 2.4.6, Prices > Discounts array contains gift cards as well. First check with PO whats the expected behavior.
NOTE. - We have cypress test which is failing
venia-integration-tests/src/tests/e2eTests/cart/userRemoveCartPromotions.spec.js
Related Issue
Closes #https://jira.corp.adobe.com/browse/PWA-3071
Acceptance
Verification Stakeholders
Specification
Verification Steps
Test scenario(s) for direct fix/feature
Test scenario(s) for any existing impacted features/areas
Test scenario(s) for any Magento Backend Supported Configurations
Is Browser/Device testing needed?
Any ad-hoc/edge case scenarios that need to be considered?
Screenshots / Screen Captures (if appropriate)
Breaking Changes (if any)
Checklist