perslev / CanvasSync

Synchronise modules, assignments and files located on a Canvas by Instructure web server to your local computer.
MIT License
63 stars 28 forks source link

add fix to ignore restricted courses #23

Closed sd5969 closed 3 years ago

sd5969 commented 3 years ago

Hi @perslev!

I noticed an issue with CanvasSync failing when I had courses with access_restricted_by_date set as True - syncing files, updating settings, etc. would crash. For these courses, the normally available fields like course_name, etc. were unavailable, causing the app to crash. I added a fix in the base API to ignore these restricted access courses. Let me know if you'd want to implement this a different way, otherwise feel free to merge!

Sanjit

perslev commented 3 years ago

Looks good, thanks!