Skip to content
This repository was archived by the owner on Apr 9, 2024. It is now read-only.

refactor(rpc): change the get balance interface #59

Merged
merged 16 commits into from
Jul 16, 2021
Merged

refactor(rpc): change the get balance interface #59

merged 16 commits into from
Jul 16, 2021

Conversation

KaoImin
Copy link

@KaoImin KaoImin commented Jul 14, 2021

What this PR does / why we need it:

  1. Change the argument of the get_balance interface to a struct.
  2. Change the udt_hashes to a HashSet, if it is empty, means to get the balance of all assets.

Which issue(s) this PR fixes:

Fixes #

Which docs this PR relation:

Ref #

Which toolchain this PR adaption:

No Breaking Change

Special notes for your reviewer:

@KaoImin KaoImin requested review from rev-chaos and bitrocks July 14, 2021 09:38
Eason Gao and others added 2 commits July 16, 2021 16:11
udt_hashes: Vec<Option<H256>>,
udt_hashes: HashSet<Option<H256>>,
address: QueryAddress,
block_number: Option<u64>,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

// Not support yet, so we should return error, when the block_number is none, which is missed now

@KaoImin KaoImin merged commit 2663391 into main Jul 16, 2021
@KaoImin KaoImin deleted the generic branch July 16, 2021 08:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants