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.

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

Enhancements to the Admin dashboard page #532

Open
utsab opened this issue Dec 26, 2024 · 2 comments
Open

Enhancements to the Admin dashboard page #532

utsab opened this issue Dec 26, 2024 · 2 comments
Labels
CTI good first issue Good for newcomers

Comments

@utsab
Copy link
Collaborator

utsab commented Dec 26, 2024

This issue is related to Issue #467 and PR #468. The referenced PR did successfully refactor the table on the admin page to use react-table. However, the PR introduced a few discrepancies compared to the original design of the admin table.

Original admin table:

Image

New admin table (after PR 468) :

Image

The goal of this issue is to fix the following discrepancies:

  1. The new admin table is missing a footer where the user can change the number of entries per page
  2. The "UserEmail" column should be titled "Email"
  3. The styling has changed. If possible, use the previous styling where there was no border surrounding the table. There also should be a visible line separating each row.

Steps to recreate the issue:

  1. Make sure you can successfully log in via Github (use the Github sign-in option, not Auth0)
  2. Run "npx prisma studio" in the terminal, navigate to the "Users" table, change the role of your user to "ADMIN"
  3. Make sure you click "Save changes" in prisma studio
  4. Sign out of the Classroom app and then sign back in
  5. Navigate to the /admin page (type this into the browser)
@utsab utsab added good first issue Good for newcomers CTI labels Dec 26, 2024
@Abiral-2724
Copy link

hello @utsab
Can you please assign this to me

@utsab
Copy link
Collaborator Author

utsab commented Dec 30, 2024

@Abiral-2724 -- Sorry I have already assigned this issue to one our student teams at CTI (Computing Talent Initiative). I am planning to open up a few more issues in the coming month for non-CTI students. Please stay tuned for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CTI good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants