-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Guest print order form fixes #29309
Guest print order form fixes #29309
Conversation
@magento run all tests |
Hi @kodubovik: Thank you for your contribution. Please solve the problem about static test. |
@magento create issue |
@magento run all tests |
989615f
to
d2c368b
Compare
@magento run all tests |
Hi @VladimirZaets, thank you for the review.
|
✔️ QA Passed Manual testing scenario:
Try to search for an order by email: Before: ✖️ You entered incorrect data. Please try again After: ✔️ Order information was successfully displayed Try to search for an order by zip code: After: ✔️ Order information was successfully displayed Also, was tested Orders and Returns widget and everything works as expected. |
@magento run all tests |
@magento run Functional Tests CE |
@magento run all tests |
@magento run Functional Tests CE |
@magento run all tests |
@magento run all tests |
@magento run Functional Tests CE |
… bug/guest-print-order-form-input-trim-spaces
@magento run all tests |
Hi, @kodubovik. Can you please resolve the conflict? |
… bug/guest-print-order-form-input-trim-spaces � Conflicts: � app/code/Magento/Store/Test/Mftf/Test/StorefrontAddStoreCodeInUrlTest.xml
@magento run all tests |
Hi @kodubovik, thank you for your contribution! |
Description (*)
This pull request fixes an issue with guest print order form when a customer inputs data with trailing/leading spaces. The expected result is that spaces are trimmed and order printed successfully.
It also improves Unit tests and Functional tests to cover cases when zip code is used to search for an order on the print guest order page.
Manual testing scenarios (*)
In all cases, you should be able to print the order.
Related Pull Requests
#30102
Contribution checklist (*)
Resolved issues: