-
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
fixed disabled guest checkout issue in case of downloadable product #23972
Conversation
Hi @rani-webkul. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
@rani-webkul apparently, some tests have to be adapted, too, for this to work. Please fix those, and I see no problem in merging this. Thank you! |
@rani-webkul Awesome, thanks. |
Hi @josefbehr, thank you for the review.
|
✔️ QA Passed |
app/code/Magento/Downloadable/Observer/IsAllowedGuestCheckoutObserver.php
Outdated
Show resolved
Hide resolved
@magento run all tests |
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.
✔ Approved.
Failing tests looks not related to changes form this PR.
Hi @ihor-sviziev, thank you for the review. |
✔️ QA Passed What was tested:
Manual testing scenario:
Before: ✖️ Guest checkout is enabled After: ✔️ Guest checkout is disabled 2. Also checked the additional cases:
Manual testing scenario:
Actual Result: ✔️ Guest checkout is enabled
Manual testing scenario:
Actual Result: ✔️ Guest checkout is disabled
Manual testing scenario:
Actual Result: ✔️ Guest checkout is disabled |
@magento run all tests |
Hi @rani-webkul, thank you for your contribution! |
fixed disabled guest checkout issue in case of downloadable product #23971
Description (*)
Related Pull Requests
Fixed Issues (if relevant)
Manual testing scenarios (*)
Questions or comments
Contribution checklist (*)