search

tl-its-umich-edu / canvas-course-manager-next

Canvas Course Manager Next: A redesign of the existing CCM application. It extends Canvas features, makes cumbersome features easier to use, and adds new features.
8 stars 9 forks source link

Add Non-UM User: include found user's name on the screen #387

Closed melindakraft closed 2 years ago

melindakraft commented 2 years ago

When adding a non-UM user with the one at a time option and the user already exists, present that user's first and last name on the scree along with the "user found" message. This will help the tool user to make sure the external email address they entered was the right one and aligns with how the old CCM tool worked.

ssciolla commented 2 years ago

@show981111, here are the different steps I see for this problem:

1) Adjust the service and controller methods (and types) in APIService to return an object like this:

{
  loginId: string
  name: string
}

2) Change the return type in the frontend in api.ts so that it returns the new object. 2) Adjust the AddNonUMUsers.tsx component (and the necessary child components) so that the name is rendered.

melindakraft commented 1 year ago

Tested with existing user. Name was returned.