canvasapi
canvasapi copied to clipboard
fix: don't override `per_page` in `PaginatedList`
Proposed Changes
- Don't override
per_pageparameter - update readme to state 100 is used instead of the 10 default when it is not specified, but we specify it already by default
- add _kwargs as arg to
PaginatedListand change everykwargs=combine_kwargs(**kwargs)to_kwargs=combine_kwargs(**kwargs), since I couldn't find a reason why some were different in the documentation and commits. - specify the per_page parameter in the mock requests.
Fixes #685 .
TODO: a test maybe?
fixtures were wrong, updated them and the tests accordingly.