search

torreyleonard / algotrader

Simple algorithmic stock and option trading for Node.js.
https://www.npmjs.com/package/algotrader
Apache License 2.0
635 stars 119 forks source link

Auth POST requires a supplied device token #24

Closed Gillinghammer closed 4 years ago

Gillinghammer commented 5 years ago

I found this out when my app crashed today. Found this thread on Stack Overflow and made the adjustments.

You need to view source code when logged in to the web version of Robinhood. https://stackoverflow.com/questions/55902967/is-there-any-way-to-get-a-bearer-token-now-since-robinhood-has-changed-the-api/55941425#55941425

yorch commented 4 years ago

Your PR changes the indentation for the whole file from tabs to 2 spaces, which makes it difficult to see what actually changed. There is an .editorconfig file that tells the IDEs / Text Editor how to format certain files. If you use something like VSCode, Sublime, Atom, WebStorm or others, you can find extensions / plugins to support editorconfig files.

torreyleonard commented 4 years ago

@Gillinghammer this is great! Any chance you can make the fixes @yorch mentioned? Otherwise it's hard to see what exactly has changed and creates an inconsistency with other source files.

torreyleonard commented 4 years ago

Merged in #34