-
Notifications
You must be signed in to change notification settings - Fork 1
Enable admins to manage project status "cul-de-sacs" #182
Comments
Setting up the following admin-only actions, based on Project status (I've already added the ones in bold):
@jmimico thoughts about those additions? Furthermore, perhaps for a separate ticket: do we want to add a UI control to ask admins whether they want to send/skip emailing the users for each action? it may prove useful... I'm thinking of those times we want to change projects' status without sending alarming emails lol ;) |
Changed scope of tickets to Projects only. User management to be handled in an existing, much older ticket #7 |
@caravinci - I agree, I think the 3 actions you implemented should send e-mails to inform the user, both because it is the right thing to do and for consistency with our other actions. However, want to hear what @jmimico thinks. Additionally, I don't think I can test these since I am not an Admin on lab-enrolment. I can't see the new action buttons for projects in each of those states :(. Either I will need to be made Admin or @jmimico will need to test this. |
Got admin access and tested in lab:
@jmimico @caravinci - Tested the scope of this ticket (just the bold actions) and it works fine for me. Will let @jmimico test and confirm himself too, but I think it's ready for PR review. I think we should add the e-mail notifications also, but we can address as separate ticket that @caravinci can pickup later if he needs to move onto other projects after this week. |
Tested and appears to work as designed |
Ticket intended to "undeny", "unapprove", and "unterminate" projects and users.
Caveat: Purged data cannot be restored.
The text was updated successfully, but these errors were encountered: