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
[ Wishlist] Fixing the default value when wishlist item quantity isn't defined #29904
[ Wishlist] Fixing the default value when wishlist item quantity isn't defined #29904
Conversation
Hi @eduard13. Thank you for your contribution
❗ Automated tests can be triggered manually with an appropriate comment:
You can find more information about the builds here ℹ️ Please run only needed test builds instead of all when developing. Please run all test builds before sending your PR for review. For more details, please, review the Magento Contributor Guide documentation. 🕙 You can find the schedule on the Magento Community Calendar page. 📞 The triage of Pull Requests happens in the queue order. If you want to speed up the delivery of your contribution, please join the Community Contributions Triage session to discuss the appropriate ticket. 🎥 You can find the recording of the previous Community Contributions Triage on the Magento Youtube Channel ✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel |
Hi @eduard13. Should this PR be delivered with magento/partners-magento2ee#248? |
Hi @lenaorobei, there is no strict requirement, we can deliver it even before if possible. |
I see that WebAPI Tests are failing and it is definitely related to this change. |
@lenaorobei the failed WebAPI test is related directly to the |
Hi @BarnyShergold, thank you for the review.
|
Auto-Tests: Covered |
Hi @eduard13, thank you for your contribution! |
Description (*)
This PR is a follow up of the magento/partners-magento2ee/pull/248, where the quantity may not be passed as an input argument.
Related Pull Requests
Fixed Issues (if relevant)
N/A
Manual testing scenarios (*)
Questions or comments
Contribution checklist (*)