Skip to content

feat(admin): improve user management page #1847

Merged
netomi merged 19 commits into
eclipse-openvsx:mainfrom
gnugomez:gnugomez/main/admin-users
Jun 26, 2026
Merged

feat(admin): improve user management page #1847
netomi merged 19 commits into
eclipse-openvsx:mainfrom
gnugomez:gnugomez/main/admin-users

Conversation

@gnugomez

Copy link
Copy Markdown
Member

Resolves #1786

Assisted-By: github:claude-opus-4.6, github:claude-sonnet-4.6, anthropic:claude-opus-4-7[1m]

@gnugomez gnugomez requested review from autumnfound and netomi and removed request for netomi May 19, 2026 10:14
Comment thread server/src/main/java/org/eclipse/openvsx/UserAPI.java
Comment thread server/src/main/java/org/eclipse/openvsx/json/UserAdminJson.java Outdated
@gnugomez gnugomez requested a review from autumnfound May 20, 2026 10:41
@netomi netomi force-pushed the gnugomez/main/admin-users branch from 7cd5179 to 5c92aeb Compare May 26, 2026 07:24
@netomi

netomi commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

I would prefer that there is an additional select box to query for the role, similar to searching for an extension instead of an filter that you apply on the results

@gnugomez gnugomez force-pushed the gnugomez/main/admin-users branch from 2f2ba32 to 85c3ff8 Compare June 26, 2026 07:28

@netomi netomi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, some small changes from myside:

  • rename AdminUsers and AdminUsersResult to more meaningful names
  • add an info alert when the user role change succeeded that automatically closes
  • do not set the role in the UserJson by default, there is at least one endpoint that allows any user to search for other users, and we do not want to expose the role there

@netomi netomi merged commit d6690bb into eclipse-openvsx:main Jun 26, 2026
5 checks passed
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.

Provide a user-management page in the admin dashboard

3 participants