-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rsdev 275 transfer forms on user deletion #95
Rsdev 275 transfer forms on user deletion #95
Conversation
…eted) to end of createdBy in db. first attempt at showing warning on front-end.
…ther than all of the forms created by the user being deleted. add more docs and tidy up naming.
…time so shouldn't cause any issues
…n now differ due to form ownership transfer
… tests. remove unused imports
…nce the createdBy user can be deleted. remove test which is no longer relevant
…ld when clicking the header.
…. bump core-model feature branch version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code changes look good to me and work fine on AWS build.
Functionality-wise I think there should be more changes to UI dialog that is shown to sysadmin when user has forms - the transfer part is new and has non-intuitive consequences, but I feel the new text it's easy to miss if someone has used this dialog before an no longer goes through the text.
E.g. the part explaining the forms could be more distinguished graphically, with more/shorter fragments in bold. The label of confirmation button could be changed to 'transfer forms and delete'. Something like that:
When a sysadmin deletes a user, and the user has created forms which have been used by other users to create documents, those forms will be transferred to the sysadmin so that the deletion can take place.
If this is the case, the sysadmin is show a warning when they try to delete the user that forms will be transferred under their ownership:
Also some code cleanup on the files I changed.
I'm still working on the format of the audit trail message (it should show type FORM and probably the form ID) but everything else is good to go so raising the PR now so we can begin review earlier with the release coming up.The accompanying
rspace-core-model
PR add some annotations to the Form classes, so that the relevant fields display properly in the audit trail: