-
Notifications
You must be signed in to change notification settings - Fork 903
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
Ignore certain columns by default #1459
Comments
This issue has been automatically marked as stale due to inactivity. |
Valid. I still want to implement this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We are currently using paper_trail and have billions of items in the versions table and the table is huge.
One of the reasons is that we have
touch: true
in associations and every change in the child record touches parent record and a new version record for this is created.In reality, most (all?) projects do not need to track just this column change, additionally to other columns like
lock_version
.So, I think these should be ignored by default. I propose to not change the library to ignore it, but extend the generator to also generate an initializer file for people to use in their projects and to have these columns ignored in it. Having this initializer will be also easier for users to see what can be configured and have a reasonable configuration already in place.
An alternative library (
audited
) already does this by default https://github.com/collectiveidea/audited/blob/4ffc6c14c4d1692f1ad626f1a4e95d28c968e6bf/lib/audited.rb#L38Wdyt? I can help with the PR.
The text was updated successfully, but these errors were encountered: