-
Notifications
You must be signed in to change notification settings - Fork 12
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
Epic 2191 - Update permissions #2191
Comments
@asafkaganbezgin Per our conversation today in the daily standup Teams meeting, there are actually two scenarios for removing super user permissions from an advisor position: Scenario 1: A super user redeploys. The super user is unassigned from the super user permission. However, nobody is immediately assigned to the position. In this case, the position's permission should revert to just "user" permissions. Scenario 2: A super user redeploys. Before the super user leaves, he assigns his/her incoming replacement to the position with super user permissions. The incoming replacement should not inherit the super user permissions. The rule should be: anytime a person is removed from a position that super user permissions, the position's permissions should revert to just user permissions. |
Reference to Scenario 2 above: Answer 1: Yes, if a person who is an administrator switches to another position, the administrator permissions should be added to the newly assigned position. The administrator's former position should revert to advisor permissions. Reference to Scenario 2 above: Answer 2: Yes, if a person who is a super user switches to another position, the super user permissions should be added to the newly assigned position. The super user's former position should revert to advisor permissions. Reference to Scenario 2 above: Answer 3: Yes, if a person who is an advisor switches to another position, the advisor/user permissions should be added to the newly assigned position. The advisor's former position should maintain its advisor/user permissions. The new rule should be: A person's permissions follows him/her to their next assignment (if he/she is actually assigned to another position). In all cases, the advisor position left vacant should have its permissions revert to advisor/user permissions. |
@asafkaganbezgin From the questions and answers above, super users will occasionally assign themselves to another position and lose their super user permissions. Then the super users have to contact me (usually via email) to have their super user permissions restored to their newly assigned position. I must also go to their previous assigned (now vacant) position and set the permissions to advisor/user. Also, one time my administrator colleague switched positions and lost his administrator permissions. He had to contact me to restore his administrator positions to his newly assigned position. The new rule will save super users and administrators time and also save me time. |
When moving a person into a new position, keep their old privilege in the new position. When removing a person from their old position, revoke the privilege from the old position.
Don't create rows in peoplePositions with both positionUuid and personUuid set to NULL.
Delete rows from peoplePositions that have both positionUuid and personUuid set to NULL.
Main functionality was completed in #3532. |
Remove ability of super users to edit existing location names (was Remove ability of super users to edit existing location names #1599)
Super users are arbitrarily renaming locations.
Super users cannot edit the profile of other super users (was Super users cannot edit the profile of other super users #1571)
When a super user tries to edit the advisor position of another super user, the following pink message is displayed:
"Forbidden: Exception while fetching data (/updatePosition): You do not have permissions to do this."
Both super users were assigned to the same advisor organization.
The work around is to select the "ANET User" button for the super user profile being selected. This makes the edited super user no longer a super user, but at least the position can be edited.
Incoming advisors inherit super user permissions from their predecessors (was Incoming advisors inherit super user permissions from their predecessors #2080)
When incoming advisors are assigned to their predecessor's advisor position, the incoming advisor inherits the advisor position's super user permissions (if the predecessor was a super user).
Advisors must be trained before they can have super user permissions.
There are many untrained super users.
When super users and admins are removed from their positions, the super user permissions of that position should be converted to advisor permissions no matter if there is a new assignee or not. When advisors are removed from their positions, then the permissions of that position should remain the same.
When Super users or admins change position, they should bring their permissions to the new position.
Less important, spit off into #3557:
Depending on the object type being edited and on the role of the logged in user , one may be allowed to edit only a subset of the properties of the object.
Make sure that these permission checks are now implemented also server side (if this is not done yet).
See for instance issue Update the ability of super users to modify principal organizations #36 and Change the Data Requirements for Principal Organizations #200 .
The text was updated successfully, but these errors were encountered: