Skip to content

Missing "MarginTrading" Field in V5GetInstrumentsInfoSpotItem #205

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

Open
SystemMxCoder opened this issue Mar 10, 2025 · 2 comments
Open

Missing "MarginTrading" Field in V5GetInstrumentsInfoSpotItem #205

SystemMxCoder opened this issue Mar 10, 2025 · 2 comments

Comments

@SystemMxCoder
Copy link

Hi,

I've noticed that the MarginTrading field is missing from the V5GetInstrumentsInfoSpotItem structure. Could you confirm whether this omission is intentional? If not, I'm happy to help address this issue.

Additionally, could you provide some guidance on the contribution workflow for making changes in this area? Thanks for your support!

@hirokisan
Copy link
Owner

Could you confirm whether this omission is intentional

It is not intended, because we have not been able to keep up with the changes on the bybit side.

could you provide some guidance on the contribution workflow for making changes in this area

Thank you very much. Clear guidelines do not yet exist.

For example is here.

bybit/README.md

Lines 394 to 398 in 1d7cd29

## Contributing
I would like to cover Bybit API and contributions are always welcome. The calling pattern is established, so adding new methods is relatively straightforward. See some PRs like https://github.com/hirokisan/bybit/pull/44.
To submit issues, PRs, and every other help is welcome.

We are more likely to have you create a simple PullRequest like below.

e.g. #153

@SystemMxCoder
Copy link
Author

No problem, I will shoot a PR around the weekend.

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