Task #530
closedDelete tree node with approved automatic role
100%
Description
Automatic role with approval (by role guarantee) is asynchronous. Role and tree node with this automatic role cannot be removed directly - automatic roles has to be removed at first. Add appropriate exception and warning message to FE.
Updated by Radek Tomiška almost 7 years ago
- Status changed from In Progress to Needs feedback
- Assignee changed from Radek Tomiška to Vít Švanda
- % Done changed from 0 to 90
I've added result code exception, when role and tree node with approved automatic role is deleted.
Could you make a test and review please?
Note: Remove role and tree node with automatic role could be disabled at all maybe (LRT for removing assigned automatic role to identities is executed synchronously this way, so logged user will wait ...).
Updated by Marcel Poul almost 7 years ago
Just a note: I would expect, that if the automatic role removal is not approved, the TreeNode object is removed. Regardless if the process is asynchronous or not...
Updated by Marcel Poul almost 7 years ago
Marcel Poul wrote:
Just a note: I would expect, that if the automatic role removal is not approved, the TreeNode object is removed. Regardless if the process is asynchronous or not...
This behaviour is consistent with User removal. You can remove user even though he has roles assigned.
In fact User removal is available also if the user has role and that role has the "Approve remove role" checkbox ticked. This may be a bug?
Updated by Radek Tomiška almost 7 years ago
I thing user removal is different story. We can talk about it with Zdenek.
Updated by Vít Švanda almost 7 years ago
- Assignee changed from Vít Švanda to Ondřej Kopr
Updated by Ondřej Kopr almost 7 years ago
- Status changed from Needs feedback to Resolved
- Assignee changed from Ondřej Kopr to Radek Tomiška
- % Done changed from 90 to 100
Sorry for later reply.
Thanks for change flash message. Now it's clear. I reviewed this message on develop and project.for change flash message. Now it's clear. I reviewed this message on develop and proje
Updated by Radek Tomiška almost 7 years ago
- Status changed from Resolved to Closed