-
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
issue/26526 Fix orderRepository does not check if there are no extens… #26527
issue/26526 Fix orderRepository does not check if there are no extens… #26527
Conversation
Hi @PascalBrouwers. Thank you for your contribution
For more details, please, review the Magento Contributor Guide documentation. |
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.
@PascalBrouwers nice catch! Could you please cover the changes with any type of automated tests?
Hi @PascalBrouwers can you please look at #26526 (comment)? |
31ede98
to
bb40898
Compare
@magento run all tests |
Hi @sidolov, thank you for the review.
|
… method when address method is null - static test fix.
@magento run all tests |
Hi @PascalBrouwers, thank you for your contribution! |
…ionAttributes
Description (*)
Fixed Issues (if relevant)
Manual testing scenarios (*)
Questions or comments
Contribution checklist (*)