tl-its-umich-edu / canvas

Integration scripts between ITS TL and Instructure Canvas
Other
3 stars 8 forks source link

SIS Set URL Error should be a warning or not even notify us of any email #103

Open jonespm opened 3 years ago

jonespm commented 3 years ago

Frequently near the beginning of the semester we get emails about "SIS Set URL Error". This error unlike some others is common and because Canvas trying to update MPathways and not finding the courses.

The source of the problem is at https://github.com/tl-its-umich-edu/canvas/blob/dc77dfd1601f2ae2e2179239cc45c983459d1ec5/SIS_import/sis_set_url.rb#L687

Two options:

I think I remember some prior discussion where this isn't that important anymore because most students just access directly rather through MPathways. This was more useful back in the CTools days, so that leans toward just disabling the email.

lsloan commented 3 years ago

The error that prompted my question about this is from email…

Date: Mon, 21 Dec 2020 22:08:29 -0500
To: canvas-sis-data-alerts@umich.edu
Subject: https://umich.instructure.com SIS Set URL Error
Message-Id: <20201222030829.1F2D83A9@matador.dsc.umich.edu>
From: CTools Upload <ctupload@matador.dsc.umich.edu>

2020-12-21 22:00:08 -0500 set url result for section id=61684 with Canvas courseId=401131: result status=Failure and result message=Unable to find Class/Term Combination
2020-12-21 22:02:20 -0500 set url result for section id=28887 with Canvas courseId=413741: result status=Failure and result message=Unable to find Class/Term Combination
2020-12-21 22:02:23 -0500 set url result for section id=27868 with Canvas courseId=414000: result status=Failure and result message=Unable to find Class/Term Combination

When I reported this error to Marian Groce mariangr@umich.edu, Tena White tlwhite@umich.edu, and Nicole Guerrette nhurbain@umich.edu, Marian responded with the question, "Can you tell me which term these are for?"

I will try to find the data to tell her what the term is, but it would be helpful for error messages to include details like: