Feature #3340
closedForce remove roles option on recertification
100%
Updated by Ondřej Kopr 11 months ago
- % Done changed from 10 to 60
Recertification was pushed into my branch in first working version
Branch: https://git.bcvsolutions.eu/modules/recertification/-/tree/okopr/3340-force-recertification
Missing documentation, tests and review.
From implementation comes some question:- for removing was implemented new LRT - the LRT counts request not items,
- roles removing is synchronous,
- roles remove execution can be setup via LRT with "after date" configuration,
- force recertification was translated in CZ: Tvrdá recertifikace, EN: Hard recertification,
- LRT for removing isn't initialized by default,
- recertification request is editable after start recertification (this exists also in previous versions).
Updated by Ondřej Kopr 11 months ago
After first review Vlada found issue with propagation force flag into backend.
This issue was fixed.
Commit: https://git.bcvsolutions.eu/modules/recertification/-/commit/e92af1dd74ed64711b246c0c14295554a704eac4 (branch: lcabelka/3399-merge-branch)
Updated by Ondřej Kopr 11 months ago
- Status changed from In Progress to Needs feedback
- Assignee changed from Ondřej Kopr to Luboš Čábelka
- % Done changed from 60 to 90
Tests takes me little bit longer than I except.
But there are: https://git.bcvsolutions.eu/modules/recertification/-/commit/0906fb388dd3625d71542b7009eff5222e9a0fc0 (branch: lcabelka/3399-merge-branch)
Documentation was updated: https://wiki.czechidm.com/tutorial/adm/module_recertification
Please Luboš could you provide me the feedback? Thank you.