-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Fix #5880, deletedAt compare against NOW #5897
Conversation
Hi @mickhansen @janmeier, Please have a look when you get chance. CI is clear except of one timeout error. |
Sweet! :) |
Notice that this PR is on the 4.0 branch - You are welcome to make a PR to v3 as well, then I'll publish a new version |
I am currently using my sequelize fork in my projects so I can wait till 4.0 is release. If any one else need this update in current semversion let me know and I will push it to v3 as well. |
Pull Request check-list
npm run test
ornpm run test-DIALECT
pass with this change (including linting)?Future
in the changelog?Description of change
Closes #5880 ,
deletedAt
field is compared againstCURRENT_TIMESTAMP
rather thanIS NULL
. It also check forIS NULL
in casedeletedAt
is not set at all.