Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Create Users Table Skeleton #44

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Create Users Table Skeleton #44

wants to merge 2 commits into from

Conversation

JustinScitech
Copy link
Contributor

@JustinScitech JustinScitech commented Oct 25, 2024 •
edited
Loading

Notion ticket link

Ticket Name

Implementation description

  • This task just involved building the frontend table on the user management page, that displays all the users and user information. Does one call to the GET /user endpoint to get all the users upon page load. Designed the frontend using ChakraUI components, just basic frontend.

image

Steps to test

  1. Assume a role of animal behaviourist or administrator
  2. Access the route: /admin/users
  3. Test if you can get all the users and also press the refresh button to refresh users
  4. Update the database (add users, delete users, edit user), should see if this change
    is reflected on the table

What should reviewers focus on?

  • Just ensuring that you can access and get the users in the table.

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • I have run the appropriate linter(s)
  • I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

@JustinScitech
Refactor UserManagementPage component and UserAPIClient
c79c2af 
- Add useEffect and useState to UserManagementPage component to fetch and display users
- Update UserManagementPage component UI to display a table of users
- Add fetchUsers function to UserManagementPage component to fetch users from the backend
- Create UserAPIClient module to handle API requests for user data
- Implement get function in UserAPIClient to fetch users from the backend
- Define User type in UserTypes module to represent user data
@JustinScitech
Refactor AuthService to remove email verification check
6045845
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@JustinScitech