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

fix: icq store #259

Merged
merged 2 commits into from
Jan 28, 2024
Merged

fix: icq store #259

merged 2 commits into from
Jan 28, 2024

Conversation

emidev98
Copy link
Contributor

@emidev98 emidev98 commented Jan 28, 2024

Fix the following error for the v2.9 chain upgrade:
"failed to load latest version: version of store interchainquery mismatch root store's version; expected 8781999 got 0; new stores should be added using StoreUpgrades"

Copy link

codecov bot commented Jan 28, 2024

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (afd2aac) 70.50% compared to head (c79f674) 70.63%.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff                @@
##           release/v2.9     #259      +/-   ##
================================================
+ Coverage         70.50%   70.63%   +0.12%     
================================================
  Files                45       45              
  Lines              2092     2101       +9     
================================================
+ Hits               1475     1484       +9     
+ Misses              451      450       -1     
- Partials            166      167       +1     
Files Coverage Δ
app/upgrade_handler.go 76.47% <50.00%> (-1.80%) ⬇️

... and 3 files with indirect coverage changes

@emidev98 emidev98 marked this pull request as ready for review January 28, 2024 08:18
@tuky191 tuky191 merged commit 7cbb1f5 into release/v2.9 Jan 28, 2024
7 of 8 checks passed
@emidev98 emidev98 deleted the fix/icq/store branch January 28, 2024 08:24
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.

3 participants