Skip to content

Get fee rates #5046

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
wants to merge 12 commits into
base: develop
Choose a base branch
from
Open

Get fee rates #5046

wants to merge 12 commits into from

Conversation

rizer1980
Copy link
Contributor

[core]
AccountService getDynamicTradingFeesByInstrument - add Optional category param

[okx, binance, bybit]
add/fix getDynamicTradingFeesByInstrument

[binance]
add separate resilience to Future account
add/fix stream futures api

[okx stream]
merge public and private channels in one connection

AccountService add optional param to getDynamicTradingFeesByInstrument
[binance, okx, bybit]
implement getDynamicTradingFeesByInstrument
[binance]
fix resilience for futures
merge authed and public stream services in one
fix testnet api url
fix getDynamicTradingFeesByInstrument
@rizer1980
Copy link
Contributor Author

@timmolter
somethink wrong?
Or merge after Xchange release?

rizer1980 added 5 commits May 26, 2025 20:43
AccountService - change getDynamicTradingFeesByInstrument method optional param from Object to String
AccountService - change getDynamicTradingFeesByInstrument method optional param from Object to String
…PR/get-fee-rates

# Conflicts:
#	xchange-okex/src/main/java/org/knowm/xchange/okex/OkexAdapters.java
#	xchange-stream-okex/src/test/java/info/bitrich/xchangestream/okex/OkexStreamingPublicDataIntegration.java
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