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

[Feature] - Account Information V3 #48

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

anderson-souza
Copy link

Feature - Account Information V3

Adding the REST API GET for the endpoint /fapi/v3/account

Description

This Pull Request adds the Account Information V3(USER_DATA) REST API Request.
It also provides some code example and Unit tests

Motivation

I'm adding the Account Information V3 request since the code only have the V2 implementation.

Changes Made

  • Added a new method called accountInformationV3 on UMAccount to send the request
  • Provided Unit tests for the new method
  • Provided code examples of how to use this new method

Testing

Verification 1

  1. Execute TestUMAccountInformationV3 unit tests
    • All tests must pass

Verification 2

  1. Execute mvn test on the project root folder
    • All tests and validations must pass

Verification 3

  1. Open the PrivateConfig class and set values for TESTNET_API_KEY and TESTNET_SECRET_KEY
  2. Execute the AccountInformationV3 main method
    • The request must be sent
    • The response value must be printed on the terminal

Checklist

  • My code follows the project's coding style and guidelines.
  • I have added tests that prove my feature works.
  • I have added relevant documentation.

Thank you for considering my contribution! 😊

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.

1 participant