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
Feature/subaccount service #288
Conversation
Codecov Report
@@ Coverage Diff @@
## master #288 +/- ##
==========================================
- Coverage 74.28% 73.82% -0.46%
==========================================
Files 58 59 +1
Lines 5502 5536 +34
==========================================
Hits 4087 4087
- Misses 1040 1074 +34
Partials 375 375
Continue to review full report at Codecov.
|
v2/subaccount_service.go
Outdated
} | ||
|
||
// Test send test api to check if the request is valid | ||
func (s *TransferToSubAccountService) Test(ctx context.Context, opts ...RequestOption) (err error) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @nandubatchu , I can't find the endpoint /test
from binance API, is this an official API?
If you want to test the API, it's better to add unit test for it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me remove this /test method.
@adshao I have removed the test method |
@adshao Please validate if this can be merged.