bcwik9 / robinhood-on-rails

A web dashboard for the free trading platform Robinhood using Ruby on Rails and a private API
MIT License
151 stars 53 forks source link

Return with sells taking past positions into consideration? #27

Closed cdtweb closed 6 years ago

cdtweb commented 7 years ago

Basically, I held $VSLR previously but I sold it for a gain. I just bought in again today and the return with sells is very wrong.

I destroyed my docker instance, pulled the latest repository update, and created the instance again.

Here is a screenshot of my position: https://www.dropbox.com/s/c0ccgjn8sm9zrgs/Screen%20Shot%202017-07-03%20at%2010.45.30%20AM.png?dl=0

bcwik9 commented 7 years ago

I'd only be able to debug if I saw the buy/sell orders so I could see what it's supposed to be. return with sells does account for all previous positions, which is why it's positive since you held it and sold for a gain

bcwik9 commented 6 years ago

The sell column should match the values of the robinhood app, since it does not take past positions in to consideration. I'm going to close this for now since it seems to be working correctly