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.

Sign up for GitHub

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

Jump to bottom

Fix the deserialization of some get_account responses #33

Merged
merged 3 commits into from
Sep 11, 2024
Merged

Fix the deserialization of some get_account responses #33

merged 3 commits into from
Sep 11, 2024

Commits on Sep 11, 2024

  1. Fixed get account json parsing

    @MarcoOl94 @ptagl
    MarcoOl94 authored and ptagl committed Sep 11, 2024
    Configuration menu
    Copy the full SHA
    4c98aac View commit details
    Browse the repository at this point in the history

  2. Refactored v1/struct unit tests 

    - Added the conditional configuration of tests
- Split the account deserialization test into 3 smaller tests
- Switched to raw strings to keep JSON formatting and make the content more readable
    @ptagl
    ptagl committed Sep 11, 2024
    Configuration menu
    Copy the full SHA
    7a4fc47 View commit details
    Browse the repository at this point in the history

  3. Fixed the deserialization of get_account responses 

    The deserialization of the `core_liquid_balance` wasn't taking into account that such field could be omitted.
    @ptagl
    ptagl committed Sep 11, 2024
    Configuration menu
    Copy the full SHA
    f483c55 View commit details
    Browse the repository at this point in the history