Defect #508
closed
Account reprezentation in FE if user has many roles
100%
Description
If the user has more then one role for the system e.g. LDAP, there are some UX defects on FE. (imagine 30 LDAP roles - one per group).
- password change for LDAP system - the system shows itself 30 times
* users accounts agenda - 30 rawswill be solved #243.
It is advised to check all agendas that works with accounts, if there are more of them.
Related issues
Updated by Marcel Poul over 7 years ago
- Related to Task #507: Provisioning per role added
Updated by Radek Tomiška over 7 years ago
- Related to Task #243: Identity profile - account list added
Updated by Ondřej Kopr over 7 years ago
- Description updated (diff)
- Assignee changed from Ondřej Kopr to Radek Tomiška
- % Done changed from 0 to 40
I fixed password change form, now used AccAccountManager instead of AccIdentityAccountManager. Also add ownershipt Boolean flag for AccAccountFilter + repository. PasswordChangeDto is now used AccAccount uuid instead of AccIdentityAccount uuid.
Also refactor method changePassword in AbstractProvisioningExecutor and IdentityPasswordValidateProcessor.
Fix test that use identityAccount instead of AccAccount.
Change documentation for provisioning: https://proj.bcvsolutions.eu/ngidm/doku.php?id=en:navrh:provisionig#provisioning (better describe PasswordChangeDto)
Resolve conflict and again retest...
commit:
https://github.com/bcvsolutions/CzechIdMng/commit/bb825a22a7acb34e42383743b829f83b657e93a9
Please Radek could you make a review and check password change. Thank you.
Next thing (users accounts agenda) will be solve RT.
Updated by Ondřej Kopr over 7 years ago
Just for interest: original password change not duplicated provisioning operations. AccAccounty has been found for all identityAccounts without duplicity. Provisioning was not made for duplicate AccAccounts.
Updated by Radek Tomiška over 7 years ago
- Status changed from In Progress to Resolved
- Assignee changed from Radek Tomiška to Ondřej Kopr
- % Done changed from 40 to 100
I did test and review, it works and code looks ok, thx.