search

hyperliquid-dex / hyperliquid-rust-sdk

MIT License
68 stars 49 forks source link

Add ModifyOrder api #6

Closed DenisDeFaria closed 11 months ago

DenisDeFaria commented 11 months ago

Added the ModifyOrder Api and modified the example from order_and_cancel to order_and_modify_and_cancel

DenisDeFaria commented 11 months ago

Added bulk modify api

lmlmt commented 11 months ago

@DenisDeFaria this PR seems to have caused some issues for other users of the API, i.e. cancels are broken now. Feel free to test out these changes on your end and resubmit a new PR when it's ready.

DenisDeFaria commented 11 months ago

@DenisDeFaria this PR seems to have caused some issues for other users of the API, i.e. cancels are broken now. Feel free to test out these changes on your end and resubmit a new PR when it's ready.

Could you give more details about the error, so I can quickly fix it, and if by any chance you have a way of reproducing it ?

Thanks

Nayshins commented 10 months ago

@lmlmt @DenisDeFaria I've been running this locally but have not seen any issues with canceling orders. Do you have any direct logs where this is occurring?