DukeLearningInnovation / kits

https://dukelearninginnovation.github.io/kits
GNU General Public License v3.0
37 stars 5 forks source link

Admin 'Batch add members to Sakai' messages for failures #726

Open msoupcoff opened 1 year ago

msoupcoff commented 1 year ago

Description Failures may occur for different reasons. Some of which can likely be ignored, others may need better descriptors for why they failed. Some examples are:

  1. The Sakai site has been softly deleted
  2. The user trying to be added is already in the site and/or Kit
  3. The site is connected to a combined Kit and thus unsure which roster to add user to

To Reproduce

  1. Aggregate a list of site IDs from Sakai with states as described in Description. Include a control site that should work without any above described issues
  2. Go to Kits Batch Add (https://kits.duke.edu/admin/batch_add)
  3. Enter site IDs
  4. Select user(s)
  5. Complete batch add process

Expected behavior The email delivered when kits has completed processing changes includes descriptors of why each case failed.

Screenshots Example of failures from a batch add. Most of the 'Course doesn't support this role' failures were due to site being connected to a combined kit. KitFailures

Additional context Discovered while completing https://cit56.zendesk.com/agent/tickets/13301 Discussed briefly in Teams DM between Liz & Marty on Sept 15, 2022. Teams is too dumb to allow linking to a DM so screenshot of discussion attached. Teams