Skip to content

Add Audit Logs when assigning or removing permission for a Group#662

Merged
vincent-olivert-riera merged 1 commit intoline:masterfrom
hoangpn:feature/audit_log_for_perm_change_of_group
Oct 28, 2025
Merged

Add Audit Logs when assigning or removing permission for a Group#662
vincent-olivert-riera merged 1 commit intoline:masterfrom
hoangpn:feature/audit_log_for_perm_change_of_group

Conversation

@hoangpn
Copy link
Contributor

@hoangpn hoangpn commented Oct 28, 2025

We introduced a feature that allows users to assign or remove permission for a Group. Recording these actions as Audit Logs would be useful for users. Therefore, we have linked the GroupObjectPermission model to the create_log and delete_log functions using the post_save and post_delete signals to generate audit logs when permissions are assigned to or removed from a group.
However, the post_delete signal will be temporarily disconnected when migration file number 0033 is executed. In this scenario, Promgen removes permissions automatically, so Audit Logs are not needed.

We introduced a feature that allows users to assign or remove permission for a Group.
Recording these actions as Audit Logs would be useful for users. Therefore, we have
linked the GroupObjectPermission model to the create_log and delete_log functions using
the post_save and post_delete signals to generate audit logs when permissions are assigned
to or removed from a group.
However, the post_delete signal will be temporarily disconnected when migration file
number 0033 is executed. In this scenario, Promgen removes permissions automatically, so
Audit Logs are not needed.
@vincent-olivert-riera vincent-olivert-riera merged commit af05b81 into line:master Oct 28, 2025
5 checks passed
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.

2 participants