Skip to content

Fix isAdmin to return role#276

Merged
lucywalt merged 1 commit intomainfrom
is-admin-fix
Apr 2, 2025
Merged

Fix isAdmin to return role#276
lucywalt merged 1 commit intomainfrom
is-admin-fix

Conversation

@taoTimTim
Copy link
Copy Markdown
Collaborator

@taoTimTim taoTimTim commented Apr 2, 2025

Chaning isAdmin to return the role instead of a boolean, to match the frontend and the documentation. Resolves #282

@taoTimTim taoTimTim self-assigned this Apr 2, 2025
Copy link
Copy Markdown
Collaborator

@lucywalt lucywalt left a comment

Choose a reason for hiding this comment

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

now matches frontend and effectively resolves the issue, looks great!

@lucywalt lucywalt merged commit 32f69a1 into main Apr 2, 2025
2 checks passed
@taoTimTim taoTimTim deleted the is-admin-fix branch April 2, 2025 08:56
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.

isAdmin needs to return role

2 participants