please update GetSymbolsAsync() in to use v2 not v1

[dj-jordan]

Hiya,

FYI.. Kucoin "/api/v1/symbols" is now depreciated, we need to now use v2 Refer notes at: docs.kucoin.com/#get-symbols-list-deprecated

• v2 is fully compatible with v1
• Two main differences exist between v1 and v2

1. The baseCurrency (baseAsset) is inaccurate in v1 as can be noted from the changes in the payload sample between v1 and v2 - i.e. GALAX-USDT erroneously specifies the base asset as GALA (and not GALAX) in v1
2. The API returns mixed case in v1 for some assets - e.g. aUSD and ECOx in v1, which is AUSD and ECOX in v2.

• Can we please update the call to GetSymbolsAsync() so that it uses version 2 instead of 1...
• From looking at the code, is the change as simple as adding 2 as the optional version parameter in the _baseClient.GetUri() call at approximately line 41 of KucoinClientSpotApiExchangeData.cs?

e.g. return await _baseClient.Execute<IEnumerable<KucoinSymbol>>(_baseClient.GetUri("symbols"**,2**), HttpMethod.Get, ct, parameters: parameters).ConfigureAwait(false);

Thanks! p.s. While making edits to GetSymbolsAsync, can you please also add the missing 'minFunds' response paramter as per issue #182

[dj-jordan]

Note in terms of case differences the full list of symbols with mixed case return issues see below...

In /api/v1/symbols we have:

In /api/v2/symbols we have:

[dj-jordan]

..and for completeness, the full set of erroneous Kucoin assets in v1 (red) with the corrected values shown as per v2 (green) of the api.

[dj-jordan]

Note: until GetSymbolsAsync() is updated to v2, I am using the following TSQL base to get correct base asset returns

`SELECT CASE baseasset WHEN 'OXEN' THEN 'LOKI' WHEN 'REV' THEN 'REVV' WHEN 'BSV' THEN 'BCHSV' WHEN 'WAXP' THEN 'WAX' WHEN 'GALA' THEN 'GALAX' ELSE upper(baseasset) END as baseasset

FROM kucoin_symbol_info_table`

[JKorf]

I've switched it to V2 and added the minFunds property in the latest version. Thanks for the detailed report