Skip to content
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 audit logging using simple-history package #251

Merged
merged 1 commit into from
Jun 18, 2016

Conversation

ian-ross
Copy link
Contributor

Fixes #244.

Note that this is likely to have a substantial performance impact, because it logs all changes to all models into separate history tables in the database. This is required to guarantee that we can reconstruct all the history of database entities in the future, but it will have an impact.

Also note that any new models created from now on will need to have a history field added in the same way as is done for the existing models in this PR.

@wonderchook
Copy link
Contributor

@amplifi something to be aware of related to performance. Later we are going to add more intelligent ways to track object history, but this is a temporary solution (temporary as in at least 3 months).

@ian-ross ian-ross force-pushed the feature/audit-logging branch from 79557cc to 59fb9fb Compare June 18, 2016 16:20
@ian-ross ian-ross merged commit 8c3bd95 into master Jun 18, 2016
@ian-ross ian-ross deleted the feature/audit-logging branch June 18, 2016 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants