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

added user universal transfer method to Binance Spot #313

Merged
merged 1 commit into from Oct 7, 2021
Merged

added user universal transfer method to Binance Spot #313

merged 1 commit into from Oct 7, 2021

Conversation

amaioli
Copy link
Contributor

@amaioli amaioli commented Sep 29, 2021

I've implemented the user universal transfer method according to https://binance-docs.github.io/apidocs/spot/en/#user-universal-transfer-user_data

@codecov
Copy link

codecov bot commented Sep 29, 2021

Codecov Report

Merging #313 (8c4995c) into master (1c85553) will increase coverage by 0.09%.
The diff coverage is 88.88%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #313      +/-   ##
==========================================
+ Coverage   74.42%   74.51%   +0.09%     
==========================================
  Files          57       58       +1     
  Lines        5446     5482      +36     
==========================================
+ Hits         4053     4085      +32     
- Misses       1022     1024       +2     
- Partials      371      373       +2     
Impacted Files Coverage Δ
v2/user_universal_transfer.go 88.23% <88.23%> (ø)
v2/client.go 85.16% <100.00%> (+0.12%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1c85553...8c4995c. Read the comment docs.

@brianmori
Copy link
Contributor

hi @adshao ,

could you merge this PR? 👍

@amaioli
Copy link
Contributor Author

amaioli commented Oct 5, 2021

Hi, can you please merge it?

@adshao
Copy link
Owner

adshao commented Oct 7, 2021

Hi, can you please merge it?

Sorry for the delay reply, thanks for this PR.

@adshao adshao merged commit e13d55c into adshao:master Oct 7, 2021
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.

None yet

3 participants