Finschia / finschia-sdk

A framework for building blockchains based Finschia Mainnet that is forked from cosmos-sdk
Apache License 2.0
62 stars 29 forks source link

feat: support message based proposals #1372

Closed jaeseung-bae closed 2 months ago

jaeseung-bae commented 2 months ago

Description

closes: #XXXX

Motivation and context

How has this been tested?

Screenshots (if appropriate):

Checklist:

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 42.48366% with 88 lines in your changes are missing coverage. Please review.

Project coverage is 69.50%. Comparing base (68d8468) to head (837cf59).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1372/graphs/tree.svg?width=650&height=150&src=pr&token=m16qfzIPO7&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia)](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1372?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia) ```diff @@ Coverage Diff @@ ## main #1372 +/- ## ======================================= Coverage 69.49% 69.50% ======================================= Files 672 668 -4 Lines 56159 56149 -10 ======================================= - Hits 39028 39026 -2 + Misses 14864 14856 -8 Partials 2267 2267 ``` | [Files](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1372?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia) | Coverage Δ | | |---|---|---| | [simapp/app.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1372?src=pr&el=tree&filepath=simapp%2Fapp.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-c2ltYXBwL2FwcC5nbw==) | `80.00% <100.00%> (-0.07%)` | :arrow_down: | | [x/fswap/types/fswap.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1372?src=pr&el=tree&filepath=x%2Ffswap%2Ftypes%2Ffswap.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mc3dhcC90eXBlcy9mc3dhcC5nbw==) | `6.45% <ø> (+1.45%)` | :arrow_up: | | [x/fswap/types/codec.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1372?src=pr&el=tree&filepath=x%2Ffswap%2Ftypes%2Fcodec.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mc3dhcC90eXBlcy9jb2RlYy5nbw==) | `58.82% <87.50%> (+17.64%)` | :arrow_up: | | [x/fswap/types/genesis.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1372?src=pr&el=tree&filepath=x%2Ffswap%2Ftypes%2Fgenesis.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mc3dhcC90eXBlcy9nZW5lc2lzLmdv) | `28.00% <0.00%> (-8.85%)` | :arrow_down: | | [x/fswap/keeper/msg\_server.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1372?src=pr&el=tree&filepath=x%2Ffswap%2Fkeeper%2Fmsg_server.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mc3dhcC9rZWVwZXIvbXNnX3NlcnZlci5nbw==) | `46.87% <0.00%> (-13.13%)` | :arrow_down: | | [x/fswap/types/msgs.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1372?src=pr&el=tree&filepath=x%2Ffswap%2Ftypes%2Fmsgs.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mc3dhcC90eXBlcy9tc2dzLmdv) | `0.00% <0.00%> (ø)` | | | [x/fswap/keeper/keeper.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1372?src=pr&el=tree&filepath=x%2Ffswap%2Fkeeper%2Fkeeper.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mc3dhcC9rZWVwZXIva2VlcGVyLmdv) | `51.54% <49.55%> (-2.38%)` | :arrow_down: | ... and [2 files with indirect coverage changes](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1372/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia)