Skip to content

Add Audit Logs when assigning or removing permission#633

Merged
vincent-olivert-riera merged 1 commit intoline:masterfrom
hoangpn:feature/audit_log_for_permission_change
Aug 14, 2025
Merged

Add Audit Logs when assigning or removing permission#633
vincent-olivert-riera merged 1 commit intoline:masterfrom
hoangpn:feature/audit_log_for_permission_change

Conversation

@hoangpn
Copy link
Contributor

@hoangpn hoangpn commented Aug 13, 2025

We linked the UserObjectPermission 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 user.
However, the post_save signal will be temporarily disconnected when migration file number 0029 is executed. In this scenario, Promgen assigns permissions automatically, so Audit Logs are not needed.

@hoangpn hoangpn force-pushed the feature/audit_log_for_permission_change branch from 4bd78d8 to 51acba1 Compare August 13, 2025 09:48
We linked the UserObjectPermission 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 user.
However, the post_save signal will be temporarily disconnected when migration file number
0029 is executed. In this scenario, Promgen assigns permissions automatically, so Audit
Logs are not needed.
@hoangpn hoangpn force-pushed the feature/audit_log_for_permission_change branch from 51acba1 to bb71e69 Compare August 13, 2025 09:48
@vincent-olivert-riera vincent-olivert-riera merged commit 2476a4f into line:master Aug 14, 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