add start_date=None to get_all_open_option_orders()#525
Conversation
|
@jmfernandes can you take a look of this simple PR? |
|
@jmfernandes can you take a look of this simple PR? It contains important fix. Thanks |
|
@jmfernandes can you merge this simple and important PR? |
|
For people who need this fix, please install from here: |
|
@jmfernandes can you merge this simple and important PR? |
|
Can you take a look of this simple PR, which just fix a few small things you left out from you PR 517 (thank you for the fix). This PR is more than a year now, not sure if Josh has checked it. Maybe because I never contributed to this repo before, so he didn't take a look. If you think it's ok, can you try to create a new PR, and submit for merge? Maybe Josh will take the PR from you. Thanks. |
Upstream PR jmfernandes#525 (jmfernandes/robin_stocks). option_orders_url already accepts start_date, but the public helper wasn't threading it through. Also forwards the new param via the MCP rh_get_all_open_option_orders tool. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Thanks for this contribution! It's included with test coverage in the new repo, alongside 11 other open PRs from here. If you'd like a maintained build with this change, that's where to find it. |
fix : #507
continuation of: #517
Just add start_date=None to get_all_open_option_orders()