Skip to content
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

Applying crypto DOGE fix from #220 #276

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hoggatt
Copy link

@hoggatt hoggatt commented May 8, 2021

Honestly, I'm not 100% sure this is the right fix. It should work, but it would be nice not to have another argument to enter. I think this will be confusing to some people.

I just wanted to get the ball rolling here in hopes we can figure out a good solution. Would be nice to be able to buy and sell DOGE without having to worry about these issues.

@cheekylions
Copy link

Buying by Quantity does work as long as the quantity is submitted as an integer. Even on the Robinhood app I can't buy fractional DOGE. I think it's limited on the server side, and until they open that up on their end, this will not work.

@cheekylions
Copy link

cheekylions commented May 9, 2021

Ofc. I'm not saying your script won't work. Obviously it does (I've tested it myself from the original post.) But it's a bit confusing, as you said because of having a whole 'nother argument. Is it reasonable to code in a DOGE check, or look for the return statement from Robinhood saying "Order quantity not valid." and resubmitting until it gets a proper Order payload? I'm not as fluent in these conventions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants