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 log #279

Merged
merged 5 commits into from
Aug 9, 2022
Merged

Add audit log #279

merged 5 commits into from
Aug 9, 2022

Conversation

0xAda
Copy link
Contributor

@0xAda 0xAda commented Jun 10, 2022

Adds an audit log so we can track what changes on core. Currently only tracking config and management commands, this should eventually track more changes, but this is the bare minimum.

@codecov
Copy link

codecov bot commented Jun 10, 2022

Codecov Report

Merging #279 (5ae0c17) into master (159c2fc) will increase coverage by 0.02%.
The diff coverage is 98.64%.

❗ Current head 5ae0c17 differs from pull request most recent head 94ee8cc. Consider uploading reports for the commit 94ee8cc to get more accurate results

@@            Coverage Diff             @@
##           master     #279      +/-   ##
==========================================
+ Coverage   96.90%   96.92%   +0.02%     
==========================================
  Files         111      113       +2     
  Lines        5167     5241      +74     
  Branches      281      281              
==========================================
+ Hits         5007     5080      +73     
- Misses        128      129       +1     
  Partials       32       32              
Impacted Files Coverage Δ
src/admin/models.py 93.75% <93.75%> (ø)
src/admin/serializers.py 100.00% <100.00%> (ø)
src/admin/tests.py 100.00% <100.00%> (ø)
src/admin/views.py 100.00% <100.00%> (ø)
src/config/tests.py 100.00% <100.00%> (ø)
src/config/views.py 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Member

@thebeanogamer thebeanogamer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@0xAda 0xAda enabled auto-merge (rebase) August 9, 2022 20:59
auto-merge was automatically disabled August 9, 2022 21:00

Rebase failed

@0xAda 0xAda merged commit 8e8bc1c into master Aug 9, 2022
@0xAda 0xAda deleted the feature/auditlog branch August 9, 2022 21:34
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