Add constraints to ensure integrity for users roles, organizations and grants#2893
Conversation
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #2893 +/- ##
==========================================
- Coverage 55.32% 54.31% -1.01%
==========================================
Files 126 128 +2
Lines 10783 11598 +815
==========================================
+ Hits 5966 6300 +334
- Misses 4817 5298 +481
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
9fb8b51 to
44dacfd
Compare
44dacfd to
1a0367d
Compare
atalyaalon
left a comment
There was a problem hiding this comment.
@ziv17
Looks good.
My only concern was that given the new constrains the migration will fail: FK creation can fail due to data violations.
I consulted with co-pilot (see here the conversation in the md file and the scripts created).
I ran the readonly sql queries in prod and found one violation, and fixed it. I think we're good to go.
No description provided.