search

microsoft / coe-starter-kit

Other
737 stars 216 forks source link

[CoE Starter Kit - Feature]: Add option to hide inactive user from 'Manage SRs' screen in Manage Permission app #7860

Open sanjeev40084 opened 5 months ago

sanjeev40084 commented 5 months ago

Is your feature request related to a problem? Please describe.

In Manage SRs screen of Manage Permissions app, it shows all the users with System Administrator app no matter if the user is active or not. I do see lot's of users who are no longer active in Azure but still on the list, which i also get it because the user must had SA role in Dynamics side before their account was inactive and Dynamics doesn't delete the user automatically.

Describe the solution you'd like

It would be nice to have some sort of settings to only show active or inactive user within the 'Manage SRs' screen.

Describe alternatives you've considered

No response

Additional context?

No response

AB#2579

Jenefer-Monroe commented 5 months ago

What do you mean by Active?

sanjeev40084 commented 5 months ago

@Jenefer-Monroe Active = Enabled user. I found there were couple of users who are no longer in Azure AD, who at one point had Global Admin role and got SA role in the environment. Those all users shows up with SA role in the Manage SRs page. Let me know if that makes sense.

Jenefer-Monroe commented 5 months ago

Oh it does now thank you. I think the idea is that you would use this app to remove them from the SA role as a result. Please let me know your thoughts.

sanjeev40084 commented 5 months ago

@Jenefer-Monroe yes, that is how I'm planning to use it.

Jenefer-Monroe commented 5 months ago

I think the feature here would be to link up the record to the Users table where we already identify orphan users then. Have not investigated how difficult this would be though. Can put on the backlog to see.

sanjeev40084 commented 5 months ago

@Jenefer-Monroe how about this, why not add another column with 'User Status' and show 'Enabled' or 'Disabled'. Also allow Filter capability within that column so that it is easier to filter by 'Disabled' user status. That should be good workaround too.