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

Bail out on search queries with too many results, issue error #378

Closed ssciolla closed 2 years ago

ssciolla commented 2 years ago

In the admin version of "Merge Sections", some short or imprecise course name queries can generate many many results. While the application may be able to handle these queries, the user may have to wait an excessive amount of time and ultimately have too many results to parse through. We could alternatively bail out of certain queries if too many courses match and issue an error to the user. The maximum number of courses could be configurable.


Test Plan:

  1. Configure a course with Win 2022 term
  2. Go to Merge section feature as admin
  3. Try searching '101', 'eng', 'che'
  4. You will see an error (as snackbar) saying refine your search.
  5. Test the use case with an admin in multiple subaccount. Those sub-account users are listed here
  6. Please make sure to test that this change doesn't impact the over all search by course name experience.