-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Delete statements will not be created using clear
#7861
Conversation
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.
Please provide a new test class with its own set of entities instead of modifying an existing one.
Actually, you're quite correct here. This scenario should have been there in the first place.
Yep, looks like a missing test case of the original fix :P |
I've pushed commit with better solution to fix #7758, without this regression |
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.
@ostrolucky thanks a lot for handling it. It's indeed a much better solution. I'll address these two tiny things and we're good to ship this.
@ferrastas @ostrolucky thank you very much 👍 |
Thanks for fix it @ostrolucky 🏆 |
This is just a demo of an opened issue.