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

Can't find a way to call /api/v3/ticker/price with multiple symbols #380

Closed
blarsy opened this issue Jun 10, 2022 · 5 comments
Closed

Can't find a way to call /api/v3/ticker/price with multiple symbols #380

blarsy opened this issue Jun 10, 2022 · 5 comments

Comments

@blarsy
Copy link
Contributor

blarsy commented Jun 10, 2022

I am struggling to find how to gather prices of multiple symbols (not all of them at once, as happens when providing no value to the "symbol" Api parameter). I did not find a related Github issue either.
As per https://github.com/binance/binance-spot-api-docs/blob/master/rest-api.md#symbol-price-ticker, it is possible on the Binance API's /api/v3/ticker/price endpoint through a "symbols" parameter, but I don't find it in go-binance.

Am I missing something ?

@adshao
Copy link
Owner

adshao commented Jun 27, 2022

I am struggling to find how to gather prices of multiple symbols (not all of them at once, as happens when providing no value to the "symbol" Api parameter). I did not find a related Github issue either. As per https://github.com/binance/binance-spot-api-docs/blob/master/rest-api.md#symbol-price-ticker, it is possible on the Binance API's /api/v3/ticker/price endpoint through a "symbols" parameter, but I don't find it in go-binance.

Am I missing something ?

Hi @blarsy , symbols is a new parameter introduced in spot v3 API, it's appreciated that if you could submit a pr to add it, or I'll add it later.

@blarsy
Copy link
Contributor Author

blarsy commented Jul 1, 2022

Ok, will do.
It will be my first PR, but it's never too late to increase one's awesomeness ;-)

@blarsy blarsy closed this as completed Jul 1, 2022
@blarsy blarsy reopened this Jul 1, 2022
@blarsy
Copy link
Contributor Author

blarsy commented Jul 1, 2022

Oops, wrong button :-/

@sleeyax
Copy link

sleeyax commented Feb 17, 2024

I see the PR has been merged. This issue should therefore be closed.

@xyq-c-cpp
Copy link
Collaborator

completed, close this.

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

4 participants