-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Update subtotal.phtml #39612
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
base: 2.4-develop
Are you sure you want to change the base?
Update subtotal.phtml #39612
Conversation
Hi @vkrokh. Thank you for your contribution!
Allowed build names are:
You can find more information about the builds here For more details, review the Code Contributions documentation. |
@magento create issue |
@magento run all tests |
Hi @vkrokh, Thanks for the collaboration & contribution! ✔️ QA PassedThanks. |
@magento run all tests |
@magento run all tests |
@magento run Functional Tests B2B, Static Tests, WebAPI Tests |
@magento run all tests |
@magento run Functional Tests B2B, Functional Tests CE, Functional Tests EE |
Test failures for Functional EE are inconsistent and seems to be flaky. They neither part of PR nor failing because of the PR changes. Build 1: Allure-Report-EE Build 2: Allure-Report-EE @ihor-sviziev & @engcom-Hotel As, I have fixed the static failures, requesting you for a review so that we can proceed further on this PR. Please let me know anything else needed. Thank You! |
@engcom-Hotel Thank you for reviewing the PR changes. Moving this PR in Merge In Progress as the changes are approved and the ET is already done. |
Typo at checkout subtotal
Resolved issues: