term lookup causing error

[lsloan]

Term lookup API call is not reading all of the pages from the API. It reads only the first page and stops. If the later usage of the data from the call doesn't include the term number from the app parameter, the lookup fails.

[lsloan]

This was temporarily disabled in #41.

It needs to be corrected, preferably by resolving #43 and reenabling this feature.

[ssciolla]

Hi @lsloan, this is certainly a bug and something I overlooked before. However, I'd caution you to not jump to #43 too quickly, since I'm not sure if canvasapi supports async/await.

It would be easy enough to modify AccountManager.get_term_names to use API.get_results_from_pages instead of API.get. This is already being done when fetching courses here:

tool-migration/migration/manager.py

Lines 73 to 78 in d1d2716

	term_results = await self.api.get_results_from_pages(
	f'/accounts/{self.account_id}/courses',
	params={'enrollment_term_id': term_id},
	page_size=50,
	limit=limit_for_term
	)

Wishing you the best in 2024.