Keith Stevens
Keith Stevens
Blocking on #590 before marking ready for review so I can merge.
> Good catch, I didn't catch that. Fixed now
Posted #614 as a future cleanup around messages. The types are pretty confusing and different depending on use case.
Ah geez, I saw there's a different category for this type of request. Sorry.
Did you're [Star Sem](https://github.com/fozziethebeat/S-Space/pull/20) update fix this issue? If so, can you close it?
Changes in the hal-fix branch address this issue. The issue will be cleared after the branch is merged.
Whoever does this should probably review https://next-auth.js.org/tutorials/role-based-login-strategy
Great, I'll assign it to you for the time being.
This looks great! Whenever you're ready prepare a PR and I'll review.
I was thinking that upon new account creation we'd check the user's ID against the environment variable. If that matches then we'd update their `role` value in the `User` table....