Skip to content

Toolbar role request fix#419

Merged
eguerrant merged 1 commit intomainfrom
toolbar_fix
Apr 21, 2026
Merged

Toolbar role request fix#419
eguerrant merged 1 commit intomainfrom
toolbar_fix

Conversation

@eguerrant
Copy link
Copy Markdown
Contributor

@eguerrant eguerrant commented Apr 21, 2026

Follow up to #417

Previously, the option to create a role request in the toolbar was shown to anyone. Since role requests are limited to users who own at least one role, the option to create one from the toolbar should also be limited to role owners

@eguerrant eguerrant marked this pull request as ready for review April 21, 2026 21:07
@eguerrant eguerrant merged commit 994877a into main Apr 21, 2026
6 checks passed
@eguerrant eguerrant deleted the toolbar_fix branch April 21, 2026 21:10
Comment thread src/pages/Home.tsx
const {data: groupRequestsData} = useGetGroupRequests({
queryParams: {assignee_user_id: '@me', status: 'PENDING', page: 0, per_page: 1},
});
const {data: ownedRolesData} = useGetRoles({
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Shouldn't the /users/@me endpoint also show if there are any Groups (Roles) owned by the current user?

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.

3 participants