-
Notifications
You must be signed in to change notification settings - Fork 12
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.
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
get_positions_options: Column updated_at
doesn't exist
#139
Comments
Came here to report the same issue, but I was trying to call get_contracts() Following. |
Thanks, I’ll check it out. Sounds like an api change. |
JestonBlu
added a commit
that referenced
this issue
Jan 18, 2022
JestonBlu
added a commit
that referenced
this issue
Jan 18, 2022
(GH-139): get_positions_options: Column `updated_at` doesn't exist
@fouslim @SwingBotScripts thank you both for the tip, i have committed and pushed the changes... ill push to CRAN this week. |
Great. Thanks!!
…Sent from my iPhone
On Jan 18, 2022, at 2:25 PM, Jeston ***@***.***> wrote:
@fouslim @SwingBotScripts thank you both for the tip, i have committed and pushed the changes... ill push to CRAN this week.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi @JestonBlu,
i got an error today while calling get_positions_options
below is the traceback:
I'm using the RobinHood_1.6 downloaded from CRAN
I think the error is coming from line 66 of the get_positions_options.R when you do the inner_join
it seems that the data.frame option_market_data also have an
updated_at
column.The fix I found is to remove the column
updated_at
from the data.frameoption_marker_data
before the inner_join at lines 61-64:The text was updated successfully, but these errors were encountered: