Add support for iterating over lists with a generator.#186
Add support for iterating over lists with a generator.#186tim-schilling wants to merge 2 commits intoVingtCinq:masterfrom
Conversation
|
Looking at this set of changes, it looks like a great addition. I see you did make some stylistic changes with how you handled using the |
|
Ah, sorry about the stylistic changes. I did my best to stay consistent, but I had to make some changes to make it easier to implement the changes. I did test all three variations with the lists API endpoint with my data. I'll be using this for one of my projects. I tested the pagination logic, but had to change the page_size as I don't have 1000 records for any endpoint. |
This requires a thorough review. I spotted two bugs in which the
get_allurl didn't match the other url. The bugs were that the one url didn't use the plural version.