-
-
Notifications
You must be signed in to change notification settings - Fork 469
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
add papertrail to all things #3778
Conversation
what's the rate of change of this database? by adding it to every change possible, if there's a lot of change that's adding an order of magnitude to two orders of magnitude more data to the database, as a heads up. |
It's relatively small right now. We could easily deal with a couple of orders of magnitude more data, especially since the paper trail objects are going to be insert-only (so shouldn't affect performance). |
4460ced
to
ee32a6b
Compare
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.
Looks good!
Description
Added paper trail to all models.