Skip to content

Add full User and Groups lists to Access page#3211

Open
charliepark wants to merge 44 commits into
mainfrom
full_user_group_lists_consolidated
Open

Add full User and Groups lists to Access page#3211
charliepark wants to merge 44 commits into
mainfrom
full_user_group_lists_consolidated

Conversation

@charliepark

@charliepark charliepark commented May 4, 2026

Copy link
Copy Markdown
Contributor

Currently, the Silo Access and Project Access page don't show all users, except when you open up the "Add user or group" form and access the dropdown:
Screenshot 2026-06-26 at 5 20 28 PM
Screenshot 2026-06-26 at 5 20 41 PM
Screenshot 2026-06-26 at 5 20 52 PM
Screenshot 2026-06-26 at 5 20 59 PM

In issue #2887, @askfongjojo requested a view where all users and groups would be displayed. This PR does that, adding a tabbed interface to show Groups / Users, and, then, on those pages, shows all groups and users.
Screenshot 2026-06-29 at 3 13 49 PM
Screenshot 2026-06-29 at 3 10 57 PM

From this tabbed view, an admin can make edits to a specific user / group's permissions, as before:
Screenshot 2026-06-29 at 11 48 11 AM

One ongoing situation that I want to mention is that we don't have a way to order / filter the responses from the API, and the default ordering from the API is by User ID. Instead of having pagination with 50 users per page, I have it selecting all-ish users (1,000) and then sorting them by name on the client. This deviates from the vanilla API response, but gives, I think, a more useful listing of users. I'm open to other approaches, if people have suggestions. I checked with Will about whether that 1,000 user cap would be a problem, and he said that in the far future it might be something to address, but that there are no current customers where that limit would be problematic, based on how they configure users/silos.

Closes #2887
Closes #1146
Closes #1359

@vercel

vercel Bot commented May 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
console Ready Ready Preview Jul 11, 2026 12:14am

Request Review

@charliepark charliepark marked this pull request as ready for review June 29, 2026 22:31
@david-crespo

david-crespo commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

The thing that I am nervous about here is merging the concept of the list of users with the concept of role access. The nice thing (even though it's also confusing) about the current access pages is that they only show what is in the role policy response for the silo and the project (or the fleet in the case of the fleet*). In the design in this PR, we have a user list and then we annotate it with role assignments. The problem for me in practice is that you might have a list of 100 users, but let's say zero or one or two of them has a role assigned, and everything else is done through groups. Similar, you might have 20 groups, but only one of them is the one you use for role assignments. In this situation (which I think is close to typical for our customers), the current role policy page that only shows the role assignments tells you what you want to know in a much more digestible way, and if we want to list all users and groups I'd rather it be on separate tabs that are dedicated to that. Now, we might still want to combine the two approaches — keep the policy page, and then add tabs for users and groups, but those tabs also include role assignments in the big list.

So I guess the short version is: what if we keep something like the current page as the Policy or Role Policy tab and add Users and Groups alongside it? Another open question for me is whether they should all be on the Access page or we might want to separate them in the sidebar into something like Silo Access and Users & Groups.

* Ooh, I think we added the fleet access page since this was started. Something to think about. The concept of listing all users doesn't make sense at the fleet level, so there's necessarily a mismatch there. This is made a little easier.

@david-crespo

Copy link
Copy Markdown
Collaborator

I like synthesizing the user's effective role from their group memberships, but I worry that it is doing too much API-like logic. Worth considering modifying the user list API to include group memberships for each user. It might not be a big deal.

image

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Want user and group list for silo admins v1 Access & IAM - tabs for all, users, groups Group detail page listing users

2 participants