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

[6] Query for additional balance objects #3072

Merged
merged 3 commits into from
Oct 9, 2019

Conversation

startailcoon
Copy link
Contributor

@startailcoon startailcoon commented Sep 25, 2019

API limitations are 100 results, use new ChainStore in bitshares/bitsharesjs#57
Call for additional results on dashboard to make sure we have all balance objects visible.

Due to extensive testing to find a way to handle this properly the requested hours for this task is 6 hrs.

Requested Bounty: 6 hrs

API limitations are 100 results, use new ChainStore
call for additional results on dashboard
@startailcoon
Copy link
Contributor Author

Additional information

There's a slight difference in the returns between the API calls.

The get_account_balances returns as vector< asset > balance
Returns:

  • share_type | amount
  • asset_id_type | asset_id

Source: https://doxygen.bitshares.org/classgraphene_1_1app_1_1database__api.html#af3a97a65a2b8a714c74a3e3f8d888019

The get_full_account returns balance object as vector< account_balance_object > balance
Returns:

  • account_id_type | owner
  • asset_id_type | asset_type
  • share_type | balance
  • bool | maintenance_flag = false

Source: https://doxygen.bitshares.org/classgraphene_1_1chain_1_1account__balance__object.html

@sschiessl-bcp sschiessl-bcp changed the title Query for additional balance objects [6] Query for additional balance objects Oct 9, 2019
@sschiessl-bcp sschiessl-bcp added this to the 190927 milestone Oct 9, 2019
@sschiessl-bcp
Copy link
Contributor

Affected account e.g. cryptobridge-payouts

Stefan Schiessl added 2 commits October 9, 2019 13:47
update bitsharesjs version

Signed-off-by: Stefan Schiessl <stefan.schiessl@blockchainprojectsbv.com>
@sschiessl-bcp sschiessl-bcp merged commit 0419837 into develop Oct 9, 2019
@sschiessl-bcp sschiessl-bcp deleted the 0_additionalBalanceObjects branch October 9, 2019 11:56
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