when turned off, DiscussionTopic.require_initial_post should be False instead of None about canvasapi HOT 4 CLOSED

If Canvas insists that their code is correct, then their documentation is wrong. The two disagree, so they cannot both be right. But I suppose that’s Canvas’s problem, not yours. I have pointed this out to them, which is probably all we can do short of opening a new bug report against their documentation.

from canvasapi.

I did a manual call to the discussion topics endpoint in Postman. It appears that despite their documentation, Canvas returns null for that parameter instead of false, hence why we return None. This appears to be the case in several other parameters as well, such as podcast_has_student_posts and only_graders_can_rate.

I am hesitant to add special cases to the library for these parameters. Instead, we should probably file a bug report with Canvas.

from canvasapi.

Apparently this is intended behavior: instructure/canvas-lms#1231 (comment)

from canvasapi.

I've reported this to Canvas (instructure/canvas-lms#1231). It does look like they've since corrected other erroneous null values (re: Matt's response), so once they fix this we should be able to close this issue.

from canvasapi.