Skip to content

feat: add group event history - #384

Open
rozzong wants to merge 8 commits into
mainfrom
feat/add-group-event-history
Open

feat: add group event history#384
rozzong wants to merge 8 commits into
mainfrom
feat/add-group-event-history

Conversation

@rozzong

@rozzong rozzong commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

feat: add group event history

Description

This PR introduces group event history for relevant actors.

  • Who: admins and group owners.
  • What: group-related IAM events.

I took inspiration from what was already done for password-related events.

Remarks:

  • Not all IAM events are group-related.
    IAM events don't all have group_id info, as it is stored in an unstructured way. Hence, event filtering is a 2 step process: first filter on SQL request, then on unstructured data using plain Python. This should be OK as a first approach as the volume of such events is likely to be limited.
  • Non-admin group owners cannot see their group's users.
    This yielded an issue when attempting to use the 'isOwner' prop to control the display of the button opening the History modal.

Related Issues

none

Checklist

  • I have tested the changes locally.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have ensured that my code follows the project's coding standards.

Screenshots

Modal button

  • Admin
Screenshot From 2026-09-10 16-52-03
  • Group owner

Notice the empty user list

Screenshot From 2026-09-10 17-15-45
  • Normal user
Screenshot From 2026-09-10 17-10-17

History

History looks like this:

Screenshot From 2026-09-10 16-52-51

@github-actions github-actions Bot added the feature New feature or request label Sep 10, 2026
@rozzong
rozzong marked this pull request as ready for review September 11, 2026 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant