-
Notifications
You must be signed in to change notification settings - Fork 334
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 specific permission to view audit logs #2190
Add specific permission to view audit logs #2190
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Staging preview deployment: > Success! https://flagsmith-staging-2174-add-specific-permission-to.vercel.app now points to https://flagsmith-frontend-staging-9ex4zkb2g-flagsmith.vercel.app [1s] |
Uffizzi Preview |
Production preview deployment: > Success! https://flagsmith-2174-add-specific-permission-to-view-aud.vercel.app now points to https://flagsmith-frontend-production-mnoglawrw-flagsmith.vercel.app [1s] |
@khvn26 I added all the typing as per your comments. |
Changes
Adds a specific permission at the organisation level for viewing Audit Log records.
Note that as part of this, I've had to add a new endpoint since the current endpoint gets all of the audit log records for all organisations that a user belongs to.
The new endpoint is nested beneath the organisation so that it can be filtered accordingly and permissions can be applied. This will not affect the UI since the audit log is always currently shown in the context of a project (which
always belongs to a single organisation).
How did you test this code?
Added unit tests