-
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
rewrite fix while product website update using mass action #21358
Conversation
Hi @AnshuMishra17. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
Hi @AnshuMishra17, any chance you can port this to 2.3 before we merge this to 2.2? Thanks! |
@miguelbalparda Magento Port Tool (https://porting.engcom.dev.magento.com) is saying that I can't port unmerged PR. |
Yes, since the change is pretty small in LOC you should be able to create a new PR pointing to the correct branch without much hassle. No need to close this one. Thanks! |
@miguelbalparda Can I edit the PR and change the base? |
@miguelbalparda I have created a new PR #21462 |
Hi @sidolov, thank you for the review. |
Hi @AnshuMishra17 I am not able to reproduce issue #21329 on 2.2-develop. |
@AnshuMishra17 , I am closing this PR now due to inactivity. |
Hi @AnshuMishra17, thank you for your contribution! |
Description (*)
URL rewrite fix while product website update using mass action
Fixed Issues (if relevant)
Manual testing scenarios (*)