-
Notifications
You must be signed in to change notification settings - Fork 23
can't create market or limit orders #6
Comments
Interesting, it's been a little while since ive looked at those endpoints as i didn't see much news of them changing since they last were modified. Within the next few days ill debug the situation and see whats going on. Its about time some JUnit tests were also included so other people can troubleshoot the system in its entirety. Stay tuned, we'll get this sorted out. |
Yeah, it's very possible that I made a mistake, but if possible just try to make any buy order (probably sell order too) and let me know if it works! Thanks. |
Using the steps you've provided of attaching auth token requirements to the fundamental checkups has fixed one of the critical bugs from preventing the system from making the order. Unfortunately I too am reproducing the issue with an order placement returning a code 400.
There was an issue of the instrument url not being sent along with the market request, which has been resolved. After adding in this 'price' value, I still am getting a code 400 sadly. (This time, the response is HTML and not JSON, causing a stack trace to fire. Exception is now being handled a little more informatively and gracefully) I'll have to do some surveys on other repositories wrapping the RobinhoodAPI, and see if there's any similar problems perhaps alluding to a system change. Otherwise, we'll get the internal issue sorted out as soon as I can. Tldr; Valid issue! Service doesn't like how the the request is being sent. 70% sure it's an authentication issue. I'll be able to devote some proper time to this soon. Thanks for bringing this up! |
Thank you for the quick response Conrad. If there's anything I can do to help, let me know. |
Alright. One pain in the butt later. This was a two step solution. There was indeed an issue within the library causing transactions not to go through. This has been fixed(?). The other issue was discovered while jamming queries into Curl. Apparently, if your Robinhood account is older and you haven't signed the Margin Agreement request, you must do that or else the Java library will always return a "Bad Request" until you've signed (digitally) said agreement. You can find a link to do that right here! There are a few other bugs that I'd like to fix before I push the change through, but if you'd like to go through and verify that the library is working again (after signing said agreement) you can find a compiled version of the library on pull request #7 Everything seems to be working on my system, although I've only been able to test the system to a certain extent. Unfortunately I don't have any free money to try a stock purchase with, and can only judge that the system is working based on the API responses I personally get with my account. Although I'm 80% confident that things should be all set again. I'll refrain from closing the issue until the pull request has all of the other bugfixes I'd like, and is merged into master. |
had to update the fundamentals endpoint to add auth token to meet newer requirements. Then had to move "verifytickerdata" before setOrderParameters.
even afterwards, I get 400 bad request when doing either market or limit. Can anyone else successfully do an order right now?
The text was updated successfully, but these errors were encountered: