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: update transaction documentation #419
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.
Since this we've brought back the usage of CLS. Can you update that?
I forgot about this one, should we revisit this after #330 ? (I'll finish the review of the associations folder soon) Also with sequelize/sequelize#16143 being merged now. |
Sure! I'll update it with the new info once the main PR is merged |
@WikiRik CI fails due to an issue with our Netlify deployment that I have not inspected, but the PR is ready to review |
It seems to have deployed fine; https://64989bbcdd20973b67da64a0--sequelize-site.netlify.app/ |
A branch I wrote a while ago but apparently never sent?
This documents the changes made in Sequelize 7 regarding transactions