Skip to content
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

[FR] Option to always allow subadmins to impersonate their group members #132

Closed
tomneedham opened this issue Dec 13, 2018 · 10 comments
Closed
Assignees
Milestone

Comments

@tomneedham
Copy link
Contributor

No description provided.

@tomneedham
Copy link
Contributor Author

@pmaier1

@pmaier1 pmaier1 changed the title [FR] Option to always allow subadmins to impersonate their members [FR] Option to always allow subadmins to impersonate their group members Dec 13, 2018
@pmaier1
Copy link
Contributor

pmaier1 commented Dec 13, 2018

Currently you need to specify the groups a group admin can impersonate. Do they even match with the groups the user is admin of? Doesn't give the impression. But the group admin will anyway only see the users from the groups he is admin of.

I think it would be way more useful when group admins could just impersonate the users in the groups they are admin of. Why do I need to specify the groups at all? The only use case I can think of would be that I generally want group admins to be able to impersonate their user but I have some group admins that shouldn't be able to. I think this doesn't weigh out the issue that when groups change I always need to take care of changing the specified groups.

@PVince81 can we schedule the change?

@PVince81
Copy link
Contributor

@sharidas can you estimate ?

@sharidas
Copy link
Contributor

A rough estimate would be 0.5 md - 0.7 md. @PVince81

@PVince81 PVince81 added this to the development milestone Dec 18, 2018
@sharidas
Copy link
Contributor

sharidas commented Jan 4, 2019

Let me know if this proposal looks ok

  • Here is a screenshot
    impersonate

Basically the idea I got is to have 2 options

  • To allow all sub admins to impersonate its group members.
  • To selectively allow sub admins to impersonate group member.
  • At one point of time, either one option could be availed.

@sharidas
Copy link
Contributor

sharidas commented Jan 4, 2019

Based on the above proposal #132 (comment) the PR is here #134

@tomneedham
Copy link
Contributor Author

Rather than disabling the other checkbox, it would be better to use a <input type="radio"> which allows only one toggle. This gives a better UX - otherwise with the proposal above, it is not clear to the user how to get the other checkbox to become enabled again.

@sharidas
Copy link
Contributor

sharidas commented Jan 4, 2019

Rather than disabling the other checkbox, it would be better to use a <input type="radio"> which allows only one toggle. This gives a better UX - otherwise with the proposal above, it is not clear to the user how to get the other checkbox to become enabled again.

Indeed radio button. You are right, it improves the UX. I will implement it with the radio button in the current PR. Thanks.

@sharidas
Copy link
Contributor

sharidas commented Jan 6, 2019

Updated the UI with radio buttons in the PR #134
One option which I thought would be useful:

  • When user selects the radio button to grant all group admins impersonate their user, the drop down list in the second radio button is emptied. Meaning when admin changes the mind and selects the radio button to grant limited group admins access, the group selector is empty. And admin has to start selecting one by one.

@PVince81
Copy link
Contributor

PVince81 commented Mar 6, 2019

@tomneedham this was released with impersonate 0.5.0 now, you may want to notify the customer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants