SoQL query support for get_all() about sodapy HOT 3 CLOSED

Thanks for opening this issue! You are absolutely correct, get_all does not support SoQL queries, but if you look at the get implementation and tests, you can see how an example of how to get that working.

from sodapy.

Of course! Thanks for maintaining this package, it's the foundation of some important code for my work.

Ok, I'll look at get again.

In the meantime, should I submit a pull request for my version of get_all that supports SoQL queries? Sorry, I should have been more clear in my initial message.

My "functional solution" is an expanded version of get_all that can paginate using either $limit and $offset or a $query string. The trade off is its significantly more complex than the current implementation, so I understand if its not something you want to support in the main branch.

from sodapy.

Right now, for all of the endpoints, pagination is up to the user. I think that I would like to preserve this consistency. If you want to add some docs about how to do pagination using $limit and $offset, I wouldn't be opposed to that.

from sodapy.