Skip to content

Add "get accounts" endpoint#15

Merged
alexex10 merged 6 commits intomainfrom
add-get-accounts-endpoint
Jul 29, 2024
Merged

Add "get accounts" endpoint#15
alexex10 merged 6 commits intomainfrom
add-get-accounts-endpoint

Conversation

@alexex10
Copy link
Copy Markdown
Contributor

@alexex10 alexex10 commented Jul 26, 2024 •
edited
Loading

Changes

  • Add "get accounts" endpoint (user module)
  • Add close method (should be called after interaction with the API is finished)
2024-07-26 16:29:30,362 50230 8293583872 DEBUG x10.utils.http Sending GET https://api.testnet.x10.exchange/api/v1/user/accounts
2024-07-26 16:29:31,234 50230 8293583872 INFO example_accounts Accounts: status='OK' data=[AccountModel(account_id=3004, description='Main account', account_index=0, status='ACTIVE', l2_key='***', l2_vault='***', api_keys=['***']), AccountModel(account_id=7349, description='Account 2', account_index=1, status='ACTIVE', l2_key='***', l2_vault='***', api_keys=[])] error=None pagination=None

alexex10
alexex10 commented Jul 26, 2024
View reviewed changes
Comment thread x10/perpetual/trading_client/base_module.py
alexex10
alexex10 commented Jul 26, 2024
View reviewed changes
Comment thread x10/perpetual/trading_client/user_module.py Outdated
@alexex10 alexex10 marked this pull request as ready for review July 26, 2024 12:28
ex10ded
ex10ded reviewed Jul 26, 2024
View reviewed changes
Comment thread x10/perpetual/trading_client/user_module.py Outdated
ex10ded
ex10ded previously approved these changes Jul 28, 2024
View reviewed changes
@alexex10 alexex10 merged commit 9c66593 into main Jul 29, 2024
0xsumatt pushed a commit to 0xsumatt/x10_python_sdk that referenced this pull request Mar 31, 2025
Add "get accounts" endpoint (x10xchange#15)
51b6546 
* Add Add "get accounts" endpoint

* Add "get accounts" endpoint

* Add "get accounts" endpoint

* Add "get accounts" endpoint

* Add "get accounts" endpoint
@alexex10 alexex10 deleted the add-get-accounts-endpoint branch June 23, 2025 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ex10ded ex10ded ex10ded approved these changes

@dmitrykrasovskih dmitrykrasovskih Awaiting requested review from dmitrykrasovskih

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@alexex10 @ex10ded