Skip to content

Comments

Issue 165 refactor users page#230

Open
justinliangg wants to merge 14 commits intomainfrom
issue-165-Refactor_users_page
Open

Issue 165 refactor users page#230
justinliangg wants to merge 14 commits intomainfrom
issue-165-Refactor_users_page

Conversation

@justinliangg
Copy link
Collaborator

@justinliangg justinliangg commented Feb 18, 2024

Change Summary

  • Refactored the user's service to use the staff tables for roles instead. This now allows a "role" filter for the users endpoint.
  • Have moved the auth check for the pages to a withProtected component instead of using the middleware due to an issue with accessing the database in the middleware.

Change Form

Fill this up (NA if not available). If a certain criteria is not met, can you please give a reason.

  • The pull request title has an issue number
  • The change works by "Smoke testing" or quick testing
  • The change has tests
  • The change has documentation

Other Information

  • Will look at writing automated tests but this can be merged in prior to that.
  • Added a button in the profile popover to set current user as admin. (Only shows up on dev).

@justinliangg justinliangg requested a review from dct0 February 18, 2024 13:15
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.

2 participants