torreyleonard / algotrader

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

Issue with getHistoricals #44

Open itscarlosisaac opened 4 years ago

itscarlosisaac commented 4 years ago

When trying to get the user historicals, the getHistoricals method is missing the account number in the qs object

qs: { account_number: _this.account, ......

Dhavin commented 3 years ago

Hello @itscarlosisaac , I would like to contribute to this project by solving this issue. Can I?

itscarlosisaac commented 3 years ago

Sure man, go ahead, I was working on a project but is for the moment on hold.