robinhood-unofficial / pyrh

Python Framework to make trades with the unofficial Robinhood API

https://pyrh.readthedocs.io/en/latest/

MIT License

1.78k stars 603 forks source link

bug fix securities_owned() #252

Closed jitrc closed 4 years ago

jitrc commented 4 years ago

Checklist

[ ] Added tests for changed code.
[ ] Updated documentation for changed code.
[ ] I've added a news fragment of my changes with the name, "{ISSUE_NUM}.{feature|bugfix|doc|removal|misc}""

Related Issue

Description

calling securities_owned() failed.

urls.POSITIONS + "?nonzero=true" did not work without converting urls.POSITIONS to a string first.

Another way would be to solve this is urls.POSITIONS.with_query("nonzero=true")

stale[bot] commented 4 years ago

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] commented 4 years ago

Closing this pull request automatically because it has not had any activity since it has been marked as stale. If you think it is still relevant and should be addressed, feel free to open a new one.