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
Request additional information on request course page #5521
Conversation
Here is the behavior I am getting:
|
I would like to remove the column "User name" from being displayed in "administrator/overview" and also "administrator/courseRequests". Opinions? |
The URL Line 1551 in af962cd
This code reads the data from the DB in it's SQL file here:
The new fields will need to be added there. |
That makes sense! Thank you |
Co-authored-by: Matthew West <matt@prairielearn.com>
Co-authored-by: Matthew West <matt@prairielearn.com>
@@ -33,7 +35,9 @@ | |||
<tr> | |||
<td class="align-middle"><%= req.short_name %></td> | |||
<td class="align-middle"><%= req.title %></td> | |||
<td class="align-middle"><%= req.user_name %></td> | |||
<td class="align-middle"><%= req.first_name + ' ' + req.last_name %></td> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd consider including the user.name
value here, just to see if it matches what they're self-reporting?
<td class="align-middle"><%= req.first_name + ' ' + req.last_name %></td> | |
<td class="align-middle"><%= req.first_name + ' ' + req.last_name %> (<%= req.user_name %>)</td> |
* added new fields to instructor form UI * added columns to db * changed type of email field in form * update alma message with new fields * added new parameters to course_requests_insert function and updated sprocs * updated sql code in sprocs * fixed email label * updated Admin UI with new columns * added new fields to admin course request * one more update - not sure is needed * added review updates * added new columns to admin overview sql * updated admin table to remove auth_name column * fixed migration number after merging master into branch * prettier fixes * updated the order of course_requests table * Update pages/administratorCourseRequests/courseRequestsTable.ejs Co-authored-by: Matthew West <matt@prairielearn.com> * Update pages/instructorRequestCourse/instructorRequestCourse.ejs Co-authored-by: Matthew West <matt@prairielearn.com> Co-authored-by: Matthew West <matt@prairielearn.com>
Adding more information to course request page, to facilitate the approval process.