Skip to content

129 people page#131

Merged
GauriRajesh733 merged 7 commits intomainfrom
rob-129-people-page
Apr 4, 2026
Merged

129 people page#131
GauriRajesh733 merged 7 commits intomainfrom
rob-129-people-page

Conversation

@robpatterson13
Copy link
Copy Markdown

ℹ️ Issue

Closes #129

📝 Description

Added the people page at /people, only visible to admins and volunteers. This is facilitated through a ProtectedRoute container, which is like AuthedApp but redirects unauthorized users to /archive instead of /{role}. The "People" button on the sidebar is only available to privileged users. Additionally, the "Create User" button is only visible to admin role.

Some other stuff:

  • moved auth guard from UsersController specifically to the remove user endpoint
  • refactored the sidebar in root.tsx into its own component
  • since we plan on replacing the design I had claude give serviceable css that will be replaced once we have something in figma

✔️ Verification

Screenshot 2026-03-20 at 2 50 09 PM User that isn't logged in Screenshot 2026-03-20 at 2 49 44 PM `/people` page for volunteer role Screenshot 2026-03-20 at 2 49 33 PM `/people` page for admin

🏕️ (Optional) Future Work / Notes

I'd like to refactor ProtectedRoute and AuthedApp to use the same backing logic (they're pretty much the same contained with a different final result) if we deem that its necessary

Copy link
Copy Markdown
Collaborator

@GauriRajesh733 GauriRajesh733 left a comment

Choose a reason for hiding this comment

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

lgtm!! thanks for refactoring the navigation bar too! i agree i think consolidating ProtectedRoute and AuthedApp is a good idea!

@GauriRajesh733
Copy link
Copy Markdown
Collaborator

updated page to show title and role based on user entity updates in #138
image

@GauriRajesh733 GauriRajesh733 merged commit 7259fa9 into main Apr 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

People Page

2 participants