Skip to content

Add Broker endpoints#84

Merged
JKorf merged 24 commits into
JKorf:mainfrom
Alex1911-Jiang:BrokerApi
Mar 24, 2026
Merged

Add Broker endpoints#84
JKorf merged 24 commits into
JKorf:mainfrom
Alex1911-Jiang:BrokerApi

Conversation

@Alex1911-Jiang
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Owner

@JKorf JKorf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR, could you also add some unit tests for the requests?

Comment thread Bitget.Net/Clients/BrokerApiV2/BitgetRestClientBrokerApi.cs Outdated
@Alex1911-Jiang
Copy link
Copy Markdown
Contributor Author

Done, but I'm not very familiar with writing unit tests—please let me know if anything looks off.

@Alex1911-Jiang Alex1911-Jiang requested a review from JKorf March 21, 2026 14:45
@JKorf JKorf merged commit f3a91f3 into JKorf:main Mar 24, 2026
1 check passed
@Alex1911-Jiang Alex1911-Jiang deleted the BrokerApi branch March 24, 2026 14:09
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

Successfully merging this pull request may close these issues.

2 participants