Add community filter & display to audit logs - #2161
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files
☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
cellio
left a comment
There was a problem hiding this comment.
I tested this and aside from one issue that is probably about how we set up dev environments, not about this change (see separate comment), this works as expected and is a big usability improvement. Scoping to global admins is fine.
No, I think it's better if the entry shows all the logged info even if this piece is contextually redundant. It's not like we're filling up your screen with a bunch of irrelevant stuff... |
cellio
left a comment
There was a problem hiding this comment.
Community links work as expected now. LGTM.
This PR adds a community filter & display for audit logs:
Also restores the "grant" button for making a user a local admin (seems like it recently got broken by accident)
For now the log's community is always shown - even if the user can only see the current community's logs - does anyone think we should conditionally hide it?