Add notice about moved templates during upgrade#11357
Conversation
|
However, I'm not sure, if we didn't violate or BC Policy: https://docs.sylius.com/en/latest/book/organization/backward-compatibility-promise.html#templates /cc @pamil |
|
I can add a 😘 |
pamil
left a comment
There was a problem hiding this comment.
Looks like a good hotfix. However, we should fix the root cause - those templates shouldn't just disappear in 1.7.
Using the old files leads to a not found template error in Front or Admin.
|
The base of this pull-request was changed, you need fetch and reset your local branch Unless you added new commits (to this branch) locally that you did not push yet, Feel free to ask for assistance when you get stuck 👍 |
|
Created #11365 to track the core issue. |
|
Thank you, Jacques! 🎉 |
Using the old files leads to a not found template error in Front or Admin.
This PR is quite simple. I've just added some information about the move of some templates.
I was using these templates and faced the issue in production since it was an edge case.
Better to tell than sorry.