search

AlgoTrader / betfair-sports-api

Library is discontinued
MIT License
55 stars 15 forks source link

Typos in betfair_session.js getAllMarkets invocation #4

Closed Rymez2K closed 12 years ago

Rymez2K commented 12 years ago

Line 158 if (options.contries instanceof Array) { needs changing to if (options.countries instanceof Array) {

and Line 159 request.countries = [ 'country' ].concat(options.countries); needs changing to request.countries = [ 'Country' ].concat(options.countries);

Thanks Pete

AlgoTrader commented 12 years ago

Thanks. I never tested getAllMarkets with countries filter. Will fix today.

There are more things coming with node 0.8 release. 0.8 is too early to be used for now but should be in weeks

AlgoTrader commented 12 years ago

Can you please test fix?

AlgoTrader commented 12 years ago

NPM is also there. 0.8 is not yet

Rymez2K commented 12 years ago

Great that is now working as expected. Many thanks this is a brilliant project, looking forward to 0.8