-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
feat: unscoped association (#5899) #6246
Conversation
Did I do something wrong? The test failed. SQLite: MySQL: |
I searched by
mysql seems to be related to this PR, or an existing bug has been tested. If it is confirmed that it has nothing to do with PR, you can notify me to review.
|
Got it. |
Before this pr, we need to merge #6250 first (for SQLite test) And #6127 (comment) (for MySQL test) |
By the way, how should I notify you @a631807682 .
I don't use GitHub very well. |
Both ok, because it's all about fixing unit tests. |
What did this pull request do?
delete old records when
Replace
,Delete
,Clear
associations inUnscoped
model.Of course, permanent or soft deletion depends on
DB.Unscoped
.See #5899
User Case Description