Skip to content

feat(admin): add /admin/users/@username#343

Merged
vandorena merged 1 commit into
hackclub:mainfrom
Lazylllama:admin-users-username-url
Jun 4, 2026
Lines changed: 12 additions & 2 deletionsions
Merged

feat(admin): add /admin/users/@username#343
vandorena merged 1 commit into
hackclub:mainfrom
Lazylllama:admin-users-username-url

Conversation

@Lazylllama
Copy link
Copy Markdown
Member

@Lazylllama Lazylllama commented Jun 3, 2026

what's this do?

adds the possibility to use usernames aswell as IDs in the url for users. Returns RecordNotFound (404 page) if no user is found.

/admin/users/1 = /admin/users/@UserWithID1

show it works

worky
image
image

returns 404 if no user is found just like with id lookup
image

ai?

Used claude for some guidance due to the fact that I'm very new to ruby (no edits by AI agents)

@Lazylllama
feat(admin): add /admin/users/@username

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
GPG key ID: B5690EEEBB952194
Verified
Learn about vigilant mode
Loading
Loading status checks…
ac747fe
vandorena
vandorena approved these changes Jun 4, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@vandorena vandorena left a comment

Choose a reason for hiding this comment

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

Tested and LGTM! TY!

@vandorena vandorena added this pull request to the he merge queue Jun 4, 2026
Merged via the queue into hackclub:main with commit t 73e6ad0 Jun 4, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vandorena vandorena

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

None yet

2 participants

@Lazylllama @vandorena