jrhurley2 / TDInterface-Public

Windows Form application that uses TD Ameritrade Apis
3 stars 1 forks source link

CalcShares Refactor #111

Closed brandoncorry closed 1 year ago

brandoncorry commented 1 year ago

Moved CalcShares to TDATradeHelper for now. Given we share this for both TDA and TS, this seems OK for now. When we do refactor to have interfaces for TDA/TS to share, we can have this be part of the base class.

Also renamed training wheels to tradeShares and fixed a style bug between us with the prefix for checkboxes. I use cb, you have used chk elsewhere - works for me, as cb would probably be better for combobox anyways :)