Skip to content

320 dev implement user page design#340

Open
janekamata wants to merge 15 commits intomainfrom
320-dev---implement-user-page-design
Open

320 dev implement user page design#340
janekamata wants to merge 15 commits intomainfrom
320-dev---implement-user-page-design

Conversation

@janekamata
Copy link
Contributor

@janekamata janekamata commented Feb 25, 2026

ℹ️ Issue

Closes #320

📝 Description

Restructured User page setup for new design

  1. Refactored existing components and reorganized
  2. Created new components for the row format
  3. Created new actions/store values for sorting and searching users
  4. Created search for user based on search for grant. Search for user is based on first name, last name, and email

✔️ Verification

Tested on front end. Tried approving/denying users as well as changing role and deleting active users

image image

Test Changes

If your new feature required some test to be changed or added to fit the new functionality or changes please document these changes here.

🏕️ (Optional) Future Work / Notes

  • I changed All Users to Active Users since it seemed to make more sense to show only active here
  • Do we want pagination on the users page? I had it for a bit but it looked strange with the length of the page
  • Add profile pictures once those are implemented fully? I tried to add it but it doesn't seem to be working

@janekamata janekamata self-assigned this Feb 25, 2026
@janekamata janekamata linked an issue Feb 25, 2026 that may be closed by this pull request
) : (
<Navigate to="/login" replace />
);
});
Copy link
Collaborator

Choose a reason for hiding this comment

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

If you are just a regular emp;loyee you can get to the user page by knowing the link. The check to see if you can go to the user page should be in AppRoutes.tsx

Copy link
Collaborator

Choose a reason for hiding this comment

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

Actually prob preferably in MainPage.tsx

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.

DEV - Implement User Page Design

2 participants