-
Notifications
You must be signed in to change notification settings - Fork 84
Creating default identity definitions #6952
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
Conversation
|
Deployment failed with the following error: View Documentation: https://vercel.com/docs/two-factor-authentication |
Greptile OverviewGreptile SummaryThis migration seeds the database with 4 default identity definitions ( Key changes:
Confidence Score: 5/5
Important Files ChangedFile Analysis
|
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.
1 file reviewed, no comments
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6952 +/- ##
=======================================
Coverage 87.32% 87.32%
=======================================
Files 525 525
Lines 34421 34422 +1
Branches 3960 3960
=======================================
+ Hits 30058 30059 +1
Misses 3500 3500
Partials 863 863 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
...mbic/migrations/versions/xx_2025_11_12_1430_b2c3d4e5f6a7_add_default_identity_definitions.py
Show resolved
Hide resolved
erosselli
left a comment
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.
LGTM! don't forget the changelog
Ticket ENG-1961
Description Of Changes
Creating DB migration to seed the default identity definitions:
These identity definitions will be flagged as system managed in https://ethyca.atlassian.net/browse/ENG-1844 so they cannot be deleted.
Pre-Merge Checklist
CHANGELOG.mdupdatedmaindowngrade()migration is correct and works