rterp / SumZeroTrading

A Java API for Developing Automated Trading Applications for the Equity, Futures, and Currency Markets
http://rterp.github.io/SumZeroTrading/
Other
145 stars 48 forks source link

Specify the Account #25

Open vinceyap88 opened 8 years ago

vinceyap88 commented 8 years ago

Hi, Is there a way to specify the account number before starting the trading via the API? :)Vincent Yap

rterp commented 8 years ago

hmmm, I don't think that is there yet. What are you intending to use it for? Do you have a master account with multiple sub accounts?

vinceyap88 commented 8 years ago

I intend to create a family account. I have added the account handling in InteractiveBroker.java and TraderOrder.java. You can review the diff and merge it if it is correct. InteractiveBrokersBroker.java.diff.txt TraderOrder.java.diff.txt

rterp commented 8 years ago

Changes look good @vinceyap88 If you want you can create a pull request to submit your changes to be merged into the mainline. Also, if there are any unit tests touching the InteractiveBrokersBroker and the TradeOrder classes, it would be good to test the new account number functionality.