-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Fix cms block save for specified store view, when another block with same ID is saved for 'All Store Views' #23255
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
Conversation
…same ID is saved for 'All Store Views'.
Hi @vovayatsyuk. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
Hi @sidolov, thank you for the review. |
@magento run all tests |
Can someone please clarify the status of this issue, is it going to be fixed, or is it closed? We are using the same CMS Block identifiers, to be able to use child themes, and this stopped working after the last upgrade to 2.3.3. @sidolov or @vovayatsyuk do you know or can tell? |
Hi @engcom-Foxtrot, |
@magento give me 2.4-develop instance |
Hi @ihor-sviziev. Thank you for your request. I'm working on Magento 2.4-develop instance for you |
Hi @ihor-sviziev, here is your Magento instance. |
Hi @vovayatsyuk, I just tried to reproduce this issue on Looks like this issue was fixed in c8baed5 , so I'm closing your PR. Thank yo for contribution! |
Hi @vovayatsyuk, thank you for your contribution! |
Description
Since Magento 2.3.1 I'm not able to save CMS block when another block with the same ID is saved for 'All store views'.
Manual testing scenarios
test
for 'All Store Views'test
for specific store view onlyQuestions or comments
This PR is just a git revert of the following commit: 1f743bd
If someone knows what was this commit about I'll be glad to fix it properly.
Contribution checklist (*)