-
Notifications
You must be signed in to change notification settings - Fork 14
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
Merge branch '1.3' of ezsystems/ezplatform-kernel into 4.6 #369
Conversation
…nges For more details see https://issues.ibexa.co/browse/IBX-5388 and ezsystems/ezplatform-kernel#397 Key changes: * Fixed performance issues of content updates after field definition changes * Made DefaultDataFieldStorage extend FieldStorage * [Tests] Aligned test coverage with the changes --------- Co-Authored-By: Paweł Niedzielski <[email protected]> Co-Authored-By: Andrew Longosz <[email protected]>
719ac9e
to
98b7b50
Compare
@Nattfarinn PHPStan CI has been fixed on 4.6, so here it should pass now :) |
FYI, during retests general problem with image editor and image asset was found https://issues.ibexa.co/browse/IBX-8298. It covers the flow which was important on 3.3 for these changes (ezsystems/ezplatform-kernel#397 (review)). Update: Patch from https://github.com/spatie/image-optimizer/pull/219/files resolves the above issue. ❤️ |
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.
Did not find issues during retests.
QA Approved on Ibexa Commerce 4.6.7-dev.
Only fixing CI remains.
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.
5d3160c
to
bb1f3cb
Compare
Quality Gate passedIssues Measures |
Cross merge PR of ezsystems/ezplatform-kernel#397