scottquach / Canvas-Assignments-Transfer-For-Todoist

Transfer your school assignments from Canvas to Todoist
MIT License
41 stars 19 forks source link

Does not display courses past 100 #37

Closed pennyshi closed 1 year ago

pennyshi commented 1 year ago

The courses displayed is capped at 100 and courses after the 100th are omitted and cannot be selected

stacksjb commented 1 year ago

Interesting - are you really enrolled in >100 courses? :)

Probably same pagination issue needs to be added.

pennyshi commented 1 year ago

The list shows all courses I've enrolled or audited on Canvas over five years, so yup more than 100!

stacksjb commented 1 year ago

Wow! Never met anyone in the wild with that issue - most schools delete courses after the semester is over, so you don't have all the active ones.

That said, we just need to add pagination, so I'll mark that as a bug and work to fix that at some point. Can't promise any timeframe as I am working on some other rework at the moment though.

stacksjb commented 1 year ago

@pennyshi can you try this version? I reused/added the pagination code to the select_courses your list might be massive, sorry there isn't much I can do about that if your school is keeping everything on Canvas :) easy_run.py.zip

stacksjb commented 1 year ago

If it works, then I can push to the repo, but I have no access to an account with that many courses to test :)

stacksjb commented 1 year ago

I figured out a way to test this, I'll see if I can get it working

stacksjb commented 1 year ago

Note to self: Test with artificially low return limit

stacksjb commented 1 year ago

hi @pennyshi , I was looking at Canvas today and they let you filter by state. Would only returning "active" courses fulfill the need?

(https://community.canvaslms.com/t5/Canvas-Resource-Documents/Canvas-Enrollment-Status-Comparison/ta-p/387055)

stacksjb commented 1 year ago

@stacksjb note to self to add filtering to script for only active courses.

pennyshi commented 1 year ago

hey so sorry for late reply! for the new zip file you uploaded I ran it and encountered this error

image
pennyshi commented 1 year ago

hi @pennyshi , I was looking at Canvas today and they let you filter by state. Would only returning "active" courses fulfill the need?

(https://community.canvaslms.com/t5/Canvas-Resource-Documents/Canvas-Enrollment-Status-Comparison/ta-p/387055)

I'm wondering what do you mean by this? (I don't see the option to control this, if you mean within the Canvas interface)

stacksjb commented 1 year ago

Go to your Canvas "All Courses" page (https://school.instructure.com/courses)

You should get three sections, with Current, Past, and Future enrollments.

"Active" courses would mean it will only return those in the "Current" section.

stacksjb commented 1 year ago

Hello,

I have updated the most recent version to only return courses in an "Active" state. This should resolve that issue for you in the meantime - would you let me know?

If not we can investigate adding iterations, but seems to be a pretty edge use case :)