+
Skip to content

Add audit log #279

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

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.

@0xAda 0xAda requested review from Bentechy66 and jchristgit June 10, 2022 00:46
@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 force-pushed the feature/auditlog branch from 1e1016a to 94ee8cc Compare August 9, 2022 21:29
@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
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载