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.

Sign up for GitHub

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

Jump to bottom

Additional context #736

Open

amkisko wants to merge 2 commits into collectiveidea:main from amkisko:feature/additional-context

amkisko commented Oct 29, 2024 •

edited

Loading

Done:

updated README with additional instructions
implemented context storage and merging before audit record writing
implemented test coverage for changed parts
updated generators and templates

Background:

comment:string is not enough for storing and accessing additional context
known approach is to have separate column with additional serialized data
would be nice to avoid context storage on auditable record
would be nice to have global setter to add context once per request/job
should be always optional to add more context

References:

it will have minor conflicts with Introduce support for audited_class setting per model #735
conflicts are resolved fully in https://github.com/amkisko/audited/tree/release

amkisko added 2 commits

October 29, 2024 18:03


          Implement additional context for audit record

1a16b59


          Update migration template naming in upgrade generator

08e45f5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet