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

refactor(campaign): MainnetAccountBalanceAll: omit account with empty balance #803

Merged
merged 2 commits into from
May 18, 2022

Conversation

lumtis
Copy link
Contributor

@lumtis lumtis commented May 18, 2022

Since the client would rely on this query to construct the genesis account balances and an account can't have an empty balance in the genesis. This PR modifies MainnetAccountBalanceAll to omit the mainnet account with an empty balance:

Example: if all the shares of the accounts reflect tokens that are not in the total supply of the mainnet

@codecov
Copy link

codecov bot commented May 18, 2022

Codecov Report

Merging #803 (eb7a83c) into develop (9e38ac7) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #803   +/-   ##
========================================
  Coverage    10.56%   10.56%           
========================================
  Files          315      315           
  Lines        73466    73468    +2     
========================================
+ Hits          7760     7765    +5     
+ Misses       65519    65517    -2     
+ Partials       187      186    -1     
Impacted Files Coverage Δ
x/campaign/keeper/grpc_mainnet_account.go 81.44% <100.00%> (+0.39%) ⬆️
x/campaign/simulation/store.go 84.45% <0.00%> (+2.02%) ⬆️

@lumtis lumtis merged commit b824aa5 into develop May 18, 2022
@lumtis lumtis deleted the refactor/mainnet-balances-omit-empty branch May 18, 2022 15:16
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