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

Retrieving the on-hold balance value. #306

Closed
wants to merge 8 commits into from
Closed

Conversation

wolfovik
Copy link
Contributor

Added display for the on-hold balance. A new 'on_hold' parameter has been added. Fully backward compatible. See details in README.

@wolfovik wolfovik mentioned this pull request Oct 25, 2023
@wolfovik
Copy link
Contributor Author

I just noticed that the parse_price construct is no longer needed. The bal_dict dictionary already contains all the necessary information. In my test account, it's a
<class 'dict'>: {'wallet_currency': 5, 'wallet_country': 'RU', 'wallet_state': '', 'wallet_fee': '1', 'wallet_fee_minimum': '1', 'wallet_fee_percent': '0.05', 'wallet_publisher_fee_percent_default': '0.10', 'wallet_fee_base': '0', 'wallet_balance': '232563', 'wallet_delayed_balance': '0', 'wallet_max_balance': '12500000', 'wallet_trade_max_balance': '11250000', 'success': 1, 'rwgrsn': -2}.

@wolfovik wolfovik mentioned this pull request Oct 25, 2023
bukson pushed a commit that referenced this pull request Nov 5, 2023
@bukson
Copy link
Owner

bukson commented Nov 5, 2023

Merged manually, please next time just make 1 commit and rebase it onto current master, thanks!

@bukson bukson closed this Nov 5, 2023
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.

2 participants