LykkeCity / Trading-API

Lykke Wallet API Trading Documentation. Trading API allows you to automate trading on the Lykke HFT account.
https://www.lykke.com/
Apache License 2.0
6 stars 5 forks source link

No Side_both available in privateService.proto + default value is Side_buy not "Side_both" #9

Open direktoren opened 3 years ago

direktoren commented 3 years ago

Bug Description Documentation https://lykkecity.github.io/Trading-API/#mass-cancel-orders say query parameter for Mass cancel orders when you select a side to cancel:

side | string | query | null | (Optional) Order side Buy or Sell (both sides by default).

But the privateService.proto definition say

message CancelOrdersRequest {
    string assetPairId = 1;
    Side side = 2;
}

where Side is

enum Side {
    buy = 0;
    sell = 1;
}

Issue number #1

Issue number #2

Screenshots N/A

Browser (please complete the following information): N/A

Last upstream Slate commit (run git log --author="Robert Lord" | head -n 1): Latest