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

Combine Admin Add Account Request Flow into Instructor Account Request Flow #13066

Open
ziqing26 opened this issue Apr 20, 2024 · 0 comments · May be fixed by #13124
Open

Combine Admin Add Account Request Flow into Instructor Account Request Flow #13066

ziqing26 opened this issue Apr 20, 2024 · 0 comments · May be fixed by #13124
Labels
a-UIX User Interface, User eXperience, responsiveness c.Task Other non-user-facing works, e.g. refactoring, adding tests committers only Difficult; better left for committers or more senior developers

Comments

@ziqing26
Copy link
Contributor

  • Environment: master

Description of feature/enhancement

Before we introduced Account Request Form (#11878), admin add instructors in Admin Home Page (without the approve / reject stage) and the result is displayed as follows:

telegram-cloud-photo-size-5-6116111692854311756-y

With the Account Request Form, admin can see pending account request submitted by instructors in a scrollable table. (see more in #12933)

Screen.Recording.2024-03-25.at.1.10.51.PM.mov

We want to combine the admin adding instructor flow into the instructor account request flow.

This will need:

  1. Combine the old Result table and the new PendingAccountRequest table
  2. Integrate the Approve / Reject stage for admin added request

Justification

Instead of having two separate table in admin home page for account requests added by admin and for those submitted by instructors, combining the UI into one table makes it cleaner and less confusing.

@ziqing26 ziqing26 added a-UIX User Interface, User eXperience, responsiveness committers only Difficult; better left for committers or more senior developers c.Task Other non-user-facing works, e.g. refactoring, adding tests labels Apr 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-UIX User Interface, User eXperience, responsiveness c.Task Other non-user-facing works, e.g. refactoring, adding tests committers only Difficult; better left for committers or more senior developers
Projects
None yet
1 participant