-
-
Notifications
You must be signed in to change notification settings - Fork 265
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Patch 1 #350
Patch 1 #350
Conversation
Added 3 apis to convert currency
Added interfaces props for convert functions
Added 4. Order status (USER_DATA) 5. Get Convert Trade History (USER_DATA)
added params for: 4. Order status (USER_DATA) 5. Get Convert Trade History (USER_DATA)
fixed capitalization for imported params
Various typos fixed
Hello @tiagosiebler there seems to be a problem with my merge request, could you give me a hand? |
@tiagosiebler Hello, could you give me a hand on why the circle test is not valid? |
src/main-client.ts
Outdated
acceptQuoteRequest( | ||
params: EnableConvertSubAccountParams, | ||
): Promise<any>; { | ||
return this.postPrivate('sapi/v1/convert/acceptQuote', params); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo here is breaking the build, should be:
acceptQuoteRequest( | |
params: EnableConvertSubAccountParams, | |
): Promise<any>; { | |
return this.postPrivate('sapi/v1/convert/acceptQuote', params); | |
} | |
acceptQuoteRequest( | |
params: EnableConvertSubAccountParams, | |
): Promise<any> { | |
return this.postPrivate('sapi/v1/convert/acceptQuote', params); | |
} |
src/main-client.ts
Outdated
enableConvertSubAccount( | ||
params: AcceptQuoteRequestParams | ||
): Promise<any>; { | ||
return this.postPrivate('sapi/v1/broker/subAccount/convert', params); | ||
} | ||
|
||
convertQuoteRequest( | ||
params: ConvertQuoteRequestParams, | ||
): Promise<any>; { | ||
return this.postPrivate('sapi/v1/convert/getQuote', params); | ||
} | ||
|
||
getOrderStatus( | ||
params: GetOrderStatusParams, | ||
): Promise<any>; { | ||
return this.getPrivate('sapi/v1/convert/orderStatus', params); | ||
} | ||
|
||
getConvertTradeHistory( | ||
params: GetConvertTradeHistoryParams, | ||
): Promise<any>; { | ||
return this.getPrivate('sapi/v1/convert/tradeFlow', params); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same for these
Thank you very much for the corrections, I have made the requested changes
________________________________
De: Tiago ***@***.***>
Enviado: jueves, 27 de julio de 2023 11:15 a. m.
Para: tiagosiebler/binance ***@***.***>
Cc: Martin Pelaez ***@***.***>; Author ***@***.***>
Asunto: Re: [tiagosiebler/binance] Patch 1 (PR #350)
@tiagosiebler commented on this pull request.
________________________________
In src/main-client.ts<#350 (comment)>:
+ enableConvertSubAccount(
+ params: AcceptQuoteRequestParams
+ ): Promise<any>; {
+ return this.postPrivate('sapi/v1/broker/subAccount/convert', params);
+ }
+
+ convertQuoteRequest(
+ params: ConvertQuoteRequestParams,
+ ): Promise<any>; {
+ return this.postPrivate('sapi/v1/convert/getQuote', params);
+ }
+
+ getOrderStatus(
+ params: GetOrderStatusParams,
+ ): Promise<any>; {
+ return this.getPrivate('sapi/v1/convert/orderStatus', params);
+ }
+
+ getConvertTradeHistory(
+ params: GetConvertTradeHistoryParams,
+ ): Promise<any>; {
+ return this.getPrivate('sapi/v1/convert/tradeFlow', params);
+ }
Same for these
—
Reply to this email directly, view it on GitHub<#350 (review)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/A2CW6CIRQVK6SSID3UB3RLDXSIWSHANCNFSM6AAAAAA2ZV3T4Q>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Summary
Additional Information