-
Notifications
You must be signed in to change notification settings - Fork 690
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
Introduce support for audited_class setting per model #735
base: main
Are you sure you want to change the base?
Conversation
…rrect interpolation
…nclude only relevant code
@danielmorrison Hey! These changes has been tested in production for past two weeks, no issues found so far, works quite well. |
@danielmorrison CI tests are all actually green, there is this "missing coverage" assessment, which does not return anything locally (meaning that all changes are covered), but here it returns different results depending on Rails version and database driver. |
Any suggestions how to replicate it locally? |
There might be a bug introduced due to custom serializer, we have not figured out a test case for this one yet: |
@danielmorrison I switched from |
Done:
Background: