[WIP] User roles #38
Draft
[WIP] User roles #38
+580
−2
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gradylink
requested changes
Jan 1, 2026
Comment on lines
+14
to
+26
| else { | ||
| throw createError({ | ||
| statusCode: 401, | ||
| statusMessage: "Unauthorized", | ||
| }); | ||
| } | ||
| if (!userRoles.includes("admin")) { | ||
| throw createError({ | ||
| statusCode: 403, | ||
| statusMessage: "Forbidden", | ||
| }); | ||
| } |
Member
There was a problem hiding this comment.
I don't think these will work correctly when using client-side only rendering, we should probably use middleware instead.
Contributor
Author
There was a problem hiding this comment.
I've tested this by adding ssr: false to nuxt.config.ts and the page appears to render fine client-only (for both logged in and non-logged in with the errors). I can't find anything online or in the docs about this being an issue.
https://nuxt.com/docs/4.x/getting-started/error-handling#createerror
I'll leave this unresolved until I've done more research and testing and will come back to it
Also added POST for roles, not in use yet though
UI is functional but very ugly, holding off on fleshing this out until I know what all the page content will be
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes made
Will add more PR detail closer to completion, just throwing this up for visibility in case anything fundamental needs changed early on.
Screenshots
When logged in as admin:
When not logged in (similar if logged in as non-admin, but shows 403 forbidden instead):
