Skip to content

Commit

Permalink
Remove old function from functions_list screenshot
Browse files Browse the repository at this point in the history
  • Loading branch information
rghuckins committed Apr 24, 2018
1 parent ba31b87 commit a42df8e
Showing 1 changed file with 0 additions and 0 deletions.
Binary file modified static/functions_list.png
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

2 comments on commit a42df8e

@xtego
Copy link

@xtego xtego commented on a42df8e Jun 7, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just stumbled across this code today and it's awesome. Thanks so much for your work that you put into this. However, I think i have uncovered a bug given certain scenarios using get positions and get orders. I am getting the following error : "Robinhood API request failed. 404: (line 107)"

Using google's debug tools, it looks like it's failing on a ticker that no longer exists.

For example, I used to own a position in RPRX which no longer exists in robinhood. This is causing robinhood_get_positions() and robinhood_get_orders() to error. Other functions like robinhood_get_portfolios() are working fine for me.

Unfortunately, that's about the best I can help. I'm not a good enough coder to fix it, but I was wondering if you or someone else had an easy fix to the code to skip over tickers that don't exist.

@kkgundelli
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I ran into the same issue. Please let me know if you find a solution.

Please sign in to comment.