Skip to content
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

Feature request: add support perps #8

Open
CRossel87a opened this issue Dec 30, 2024 · 1 comment
Open

Feature request: add support perps #8

CRossel87a opened this issue Dec 30, 2024 · 1 comment

Comments

@CRossel87a
Copy link

Hi,

Nice crate!

I was able to add perp support by adding an optional leverage field to the create order struct. The default leverage is 1.0 if left blank. This causes insufficient margin error.

Pulling perp portfolio also causes json deserialize error, but have not looked into the details.

@Ohkthx
Copy link
Owner

Ohkthx commented Dec 31, 2024

Hello! I'll have to check the API documentation.

The main problem for me adding support for perps/futures would be that I have no way to really test it works correctly. Maybe the sandbox supports the endpoints now but I am not enrolled to do those on production.

I can try and see why the portfolios don't work. A lot of times the API documentation will say it returns a field as a string but in reality it is an integer or float (or vice versa).

The perp portfolio fix would be great for the current 2.0.2 branch that I am slowly working on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants