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: add minted coins to balance in x/collection MsgMintFT #1105

Merged
merged 14 commits into from
Sep 1, 2023

Conversation

0Tech
Copy link
Collaborator

@0Tech 0Tech commented Aug 31, 2023

Description

This patch would fix accumulation logic of balance in MsgMintFT.

closes: #1104

Checklist:

  • I followed the contributing guidelines and code of conduct.
  • I have added a relevant changelog to CHANGELOG.md
  • I have added tests to cover my changes.
  • I have updated the documentation accordingly.
  • I have updated API documentation client/docs/swagger-ui/swagger.yaml

@0Tech 0Tech added A: bug Something isn't working C:x/collection A: State Machine Breaking Any changes that result in a different AppState given same genesisState and txList. labels Aug 31, 2023
@0Tech 0Tech added this to the v0.48.0 milestone Aug 31, 2023
@0Tech 0Tech self-assigned this Aug 31, 2023
@0Tech 0Tech changed the title fix: add minted coins to balance fix: add minted coins to balance in x/collection MsgMintFT Aug 31, 2023
@codecov
Copy link

codecov bot commented Aug 31, 2023

Codecov Report

Merging #1105 (33e3644) into rc/v0.48.0-rcx (c95052c) will increase coverage by 0.05%.
The diff coverage is 87.89%.

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                 @@
##           rc/v0.48.0-rcx    #1105      +/-   ##
==================================================
+ Coverage           69.70%   69.75%   +0.05%     
==================================================
  Files                 642      645       +3     
  Lines               67280    67468     +188     
==================================================
+ Hits                46895    47061     +166     
- Misses              18203    18219      +16     
- Partials             2182     2188       +6     
Files Changed Coverage Δ
x/collection/keeper/migrations.go 50.00% <50.00%> (ø)
x/collection/keeper/migrations/v2/store.go 81.72% <81.72%> (ø)
x/collection/keeper/migrations/v2/keys.go 100.00% <100.00%> (ø)
x/collection/keeper/supply.go 88.76% <100.00%> (ø)

... and 1 file with indirect coverage changes

@0Tech
Copy link
Collaborator Author

0Tech commented Sep 1, 2023

I've checked the migration working as expected.

Invariant relevant commits would be reverted as a result of an internal discussion.

@0Tech 0Tech marked this pull request as ready for review September 1, 2023 09:08
CHANGELOG.md Outdated Show resolved Hide resolved
@0Tech 0Tech requested a review from zemyblue September 1, 2023 10:52
@0Tech 0Tech requested a review from jaeseung-bae September 1, 2023 10:52
@0Tech 0Tech merged commit 0a27aef into Finschia:rc/v0.48.0-rcx Sep 1, 2023
@0Tech 0Tech deleted the fix-mintft branch September 1, 2023 12:01
zemyblue added a commit to zemyblue/finschia that referenced this pull request Sep 1, 2023
zemyblue added a commit to Finschia/finschia that referenced this pull request Sep 1, 2023
…p up ostracon from `449aa3148b12` to `fc29846c918d` (#262)

* deps: bump up finschia-sdk from `v0.48.0-rc2` to `20230830070349` and bump up ostracon from `449aa3148b12` to `fc29846c918d`

Signed-off-by: zemyblue <zemyblue@gmail.com>

* chore: update changelog

Signed-off-by: zemyblue <zemyblue@gmail.com>

* deps: bump up finschia-sdk to `0a27aef22921`. (apply Finschia/finschia-sdk#1105)

Signed-off-by: zemyblue <zemyblue@gmail.com>

* chore: update changelog

Signed-off-by: zemyblue <zemyblue@gmail.com>

* chore: update go.sum

Signed-off-by: zemyblue <zemyblue@gmail.com>

---------

Signed-off-by: zemyblue <zemyblue@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: bug Something isn't working A: State Machine Breaking Any changes that result in a different AppState given same genesisState and txList. C:x/collection
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants