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

feat: Project logs audit logs #810

Merged
merged 6 commits into from
Sep 15, 2023
Merged

feat: Project logs audit logs #810

merged 6 commits into from
Sep 15, 2023

Conversation

josebui
Copy link
Contributor

@josebui josebui commented Sep 12, 2023

Description

Added audit logs for:

  • Project membership added
  • Project membership updated
  • Project updated

Fixed params for:

  • Site update audit log project ID

Checklist

  • Corresponding issue has been opened
  • New tests added

Related Issues

Fixes #763
Fixes #701
Fixes #702
Fixes #551
Fixes #553

@josebui josebui self-assigned this Sep 12, 2023
@josebui josebui marked this pull request as ready for review September 12, 2023 17:56
@josebui josebui requested review from shrouxm and david-code and removed request for shrouxm September 12, 2023 17:59
Copy link
Contributor

@david-code david-code left a comment

Choose a reason for hiding this comment

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

This looks good to me! No major comments on my side.

@josebui josebui merged commit 7fe3a89 into main Sep 15, 2023
@josebui josebui deleted the feat/project-logs-audit-logs branch September 15, 2023 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants