Skip to content

add start_date=None to get_all_open_option_orders() #525

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
mw66 wants to merge 2 commits into
base: master
Choose a base branch
from
Open

add start_date=None to get_all_open_option_orders() #525

mw66 wants to merge 2 commits into from

Conversation

@mw66
Copy link

@mw66 mw66 commented Dec 26, 2024

fix : #507

continuation of: #517

@mw66
Copy link
Author

mw66 commented Dec 26, 2024

@jmfernandes @noLeash

fix : #507

continuation of: #517

@mw66
Copy link
Author

mw66 commented Jan 17, 2025

@jmfernandes can you take a look of this simple PR?

@mw66
Copy link
Author

mw66 commented Jan 21, 2025

@jmfernandes can you take a look of this simple PR?

It contains important fix.

Thanks

@mw66
Copy link
Author

mw66 commented Mar 31, 2025

@jmfernandes can you merge this simple and important PR?

@mw66
Copy link
Author

mw66 commented Mar 31, 2025 •
edited
Loading

For people who need this fix, please install from here:

pip3 install -U git+https://github.com/mw66/robin_stocks.git@fix_2

@mw66 mw66 mentioned this pull request Apr 15, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

anyone experienced rh.get_all_open_option_orders(account_number=acct) is very slow? e.g. ~2 minutes

2 participants

@mw66 @yilabs