-
Notifications
You must be signed in to change notification settings - Fork 406
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
chore: MFA Method debugging / support improvements #4989
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ |
Docker builds report
|
Uffizzi Ephemeral Environment
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4989 +/- ##
==========================================
- Coverage 97.39% 97.39% -0.01%
==========================================
Files 1191 1192 +1
Lines 41633 41652 +19
==========================================
+ Hits 40549 40567 +18
- Misses 1084 1085 +1 ☔ View full report in Codecov by Sentry. |
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.
Changes
Add created_at and updated_at to MFAMethod class. Note that I chose not to use auto_now and auto_now_add because I wanted existing records to only be marked as updated once they have in fact been updated to avoid confusion.
How did you test this code?
N/a - trivial changes