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

Make user admin #211

Merged
merged 2 commits into from
Jan 21, 2024
Merged

Make user admin #211

merged 2 commits into from
Jan 21, 2024

Conversation

Xavilien
Copy link
Member

@Xavilien Xavilien commented Jan 20, 2024

Description

Added feature that allows admins to make users into admins from the admin page, as well as remove them from admin status. Also added a status chip for admins.

Closes #208

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Run test to add admin by going to /admin → Participants → Profile → Make Admin. Upon successfully making admin, modal should close and new admin status should be reflected as a chip under the status column
  • Run test to remove admin by going to /admin → Participants → Profile → Remove Admin. Upon successfully removing admin, modal should close and admin status should be removed under the status column

Test Configuration:

  • Node.js version: 18.18.0
  • Desktop/Mobile: Macbook Air M2
  • OS: Sonoma 14.0
  • Browser: Safari

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@pm3512 pm3512 merged commit 0bfdc06 into main Jan 21, 2024
7 checks passed
@pm3512 pm3512 deleted the make-user-admin branch January 21, 2024 00:02
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.

[Feature] Add a way to make users into admins from the admin page
2 participants