-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Reference PL of #18928 for 2.3-develop #18930
Reference PL of #18928 for 2.3-develop #18930
Conversation
Hi @maheshWebkul721. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
@slavvka PR against |
@orlangur I would say PR to 2.3 should be merged before PR to 2.2. But that doesn't mean that PR to 2.2 cannot be completed and than changes are moved to 2.3 |
@slavvka , whats the status for this PR? |
Hey @maheshWebkul721 I asked you in your PR to 2.2 to cover your change with the automated test. |
@maheshWebkul721 But since this PR will be processed firstly, my ask relates to it as well. |
@maheshWebkul721 I am closing this PR since we have another PR #19191 which has better fix and it is covered with automated tests. Thank you for your collaboration! |
Reference #18928
Fixed Issue of Place an order as an guest, the displayed customer name in the admin order management is "Guest". If the guest has created an account after placing the order, it is a link that points to the customer's account management. Invoice and shipping are also affected.
Description (*)
If Order is placed by Guest user in admin Order view, Invoice view, Shipment view, displaying Customer
name as "Guest", issue was in the quote management, its only set email for the customer, Now i have it
in this pull request.
Fixed Issues (if relevant)
Manual testing scenarios (*)
Contribution checklist (*)