-
-
Notifications
You must be signed in to change notification settings - Fork 7.3k
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
Order types #22449
base: master
Are you sure you want to change the base?
Order types #22449
Conversation
samgermain
commented
May 9, 2024
•
edited
edited
@@ -875,7 +875,7 @@ export default class lykke extends Exchange { | |||
}, market); | |||
} | |||
|
|||
async cancelOrder (id: string, symbol: Str = undefined, params = {}) { | |||
async cancelOrder (id: string, symbol: Str = undefined, params = {}): Promise<Order> { |
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.
cancelOrder returns the raw response here
return await this.privateDe ...
@@ -1354,7 +1354,7 @@ export default class whitebit extends Exchange { | |||
return this.parseOrder (response); | |||
} | |||
|
|||
async cancelOrder (id: string, symbol: Str = undefined, params = {}) { | |||
async cancelOrder (id: string, symbol: Str = undefined, params = {}): Promise<Order> { |
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 here, returns unparsed response
@@ -876,7 +876,7 @@ export default class timex extends Exchange { | |||
return this.parseOrder (order, market); | |||
} | |||
|
|||
async cancelOrder (id: string, symbol: Str = undefined, params = {}) { | |||
async cancelOrder (id: string, symbol: Str = undefined, params = {}): Promise<Order> { |
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 here
@@ -477,7 +477,7 @@ export default class zaif extends Exchange { | |||
}, market); | |||
} | |||
|
|||
async cancelOrder (id: string, symbol: Str = undefined, params = {}) { | |||
async cancelOrder (id: string, symbol: Str = undefined, params = {}): Promise<Order> { |
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
@@ -1476,7 +1476,7 @@ export default class zonda extends Exchange { | |||
}); | |||
} | |||
|
|||
async cancelOrder (id: string, symbol: Str = undefined, params = {}) { | |||
async cancelOrder (id: string, symbol: Str = undefined, params = {}): Promise<Order> { |
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
@samgermain We have a lot of methods that besides the type need to be adjusted to return the proper structure, many of them just return the raw response from the exchange |
yeah I'm working on it, it's quite a few changes |