Skip to content

Commit

Permalink
feat: introduce new modules(fswap, fbridge) (Finschia#1352)
Browse files Browse the repository at this point in the history
* feat: initialize fswap module (Finschia#1336)

* feat: initial fswap module

Signed-off-by: 170210 <j170210@icloud.com>

* chore: correct dependencies

Signed-off-by: 170210 <j170210@icloud.com>

* chore: remove unuse files

Signed-off-by: 170210 <j170210@icloud.com>

* feat: add proto files

Signed-off-by: 170210 <j170210@icloud.com>

* chore: make proto-gen

Signed-off-by: 170210 <j170210@icloud.com>

* chore: fix compile error

Signed-off-by: 170210 <j170210@icloud.com>

* chore: add new field in params proto

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix lint

Signed-off-by: 170210 <j170210@icloud.com>

* feat: add query&tx commands of fswap module

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix for comment

Signed-off-by: 170210 <j170210@icloud.com>

* docs: update CHANGLOG.md

Signed-off-by: 170210 <j170210@icloud.com>

* docs: fix error docs

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix consensusversion

Signed-off-by: 170210 <j170210@icloud.com>

* fix: remove route,type&getSignBytes in msgs.go

Signed-off-by: 170210 <j170210@icloud.com>

* fix: update proto files

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix NewKeeper

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix for comment

Signed-off-by: 170210 <j170210@icloud.com>

---------

Signed-off-by: 170210 <j170210@icloud.com>

* feat: implement fswap module's genesis and params (Finschia#1339)

* feat: implement genesis related functions

Signed-off-by: 170210 <j170210@icloud.com>

* chore: use mockgen

Signed-off-by: 170210 <j170210@icloud.com>

* chore: add tests

Signed-off-by: 170210 <j170210@icloud.com>

* fix: add subspace in paramsKeeper

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix lint

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix for test errors

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix lint

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix appmodulebasic

Signed-off-by: 170210 <j170210@icloud.com>

* chore: update CHANGLOG.md

Signed-off-by: 170210 <j170210@icloud.com>

* fix: use configuration instead of params

Signed-off-by: 170210 <j170210@icloud.com>

* chore: refactor new&default fswapped

Signed-off-by: 170210 <j170210@icloud.com>

* fix: add total_supply into genesisStatus & remove calculate logic

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix lint

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix module_simulation

Signed-off-by: 170210 <j170210@icloud.com>

* fix: set swappableNewCoinDenom as params

Signed-off-by: 170210 <j170210@icloud.com>

* fix: change total_supply name to swappableNewCoinAmount

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix lint

Signed-off-by: 170210 <j170210@icloud.com>

* fix: use io/collection to generate key

Signed-off-by: 170210 <j170210@icloud.com>

* Revert "fix: use io/collection to generate key"

This reverts commit e42e064.

* fix: change sdk.Coin to sdk.Int

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix lint

Signed-off-by: 170210 <j170210@icloud.com>

* fix: remote unnecessary codes

Signed-off-by: 170210 <j170210@icloud.com>

* fix: add validate for swappable new coin

Signed-off-by: 170210 <j170210@icloud.com>

* fix: add randomizedGenState

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix error doc

Signed-off-by: 170210 <j170210@icloud.com>

* fix: fix lint

Signed-off-by: 170210 <j170210@icloud.com>

* fix: change proto message fields' name

Signed-off-by: 170210 <j170210@icloud.com>

---------

Signed-off-by: 170210 <j170210@icloud.com>

* feat: initialize fbridge module (Finschia#1340)

* initialize fbridge module

* add all proto files for fbridge

* add missing proto files

* initialize module structure

* modify proto based on discussion

* add codec

* add CHANGELOG

* modify UpdateRole feature

* feat: fbridge transfer (Finschia#1347)

* impl bridge transfer tx/query

* add CLI

* feat: save sequence for every bridge request (Finschia#1351)

* feat: add SeqToBlocknum

* add CHANGELOG

* add query bound for SeqToBlocknums

* feat: fswap module's basic functionality(MsgSwap, MsgSwapAll, Query, Proposal) (Finschia#1345)

* refactor: fswap module's proto

Signed-off-by: 170210 <j170210@icloud.com>

* refactor: remove config&params and modify genesis

Signed-off-by: 170210 <j170210@icloud.com>

* refactor: use gov/proposal to change fswapInit

Signed-off-by: 170210 <j170210@icloud.com>

* chore: fix simulation

Signed-off-by: 170210 <j170210@icloud.com>

* chore: correct query cli

Signed-off-by: 170210 <j170210@icloud.com>

* chore: add query for test

Signed-off-by: 170210 <j170210@icloud.com>

* chore: lint fix

* chore: fix for tx cli

* chore: update proto

* chore: update genesis logic

* chore: add basic tx and queries

* chore: rename fswap to swap

* chore: change old/new to from/to

* chore: clean-up

* chore: clean errors and rename

* chore: update proto

* chore: clean unnecessary member variables

* chore: update proto, introduce config, clean code

* chore: remove redundant denom validation

* chore: remove cache-like operation, rename swap proposal in proto, errors start with number 2

* chore: apply feedbacks

* chore: fix some

* chore: fix for initGenesis

* chore: apply feedbacks

* chore: update changelog

* chore: remove redundant logic

* chore: update error doc

* chore: update proto and logic to accept bank.Metadata

* chore: add query to get all the swaps

* chore: change swap_multiple to swap_rate

* chore: disallow updating existing denom metadata with different values

* chore: fix for calcSwap logic

* chore: extract CalcSwap to function

* chore: add missing query cmd

* chore: fix for proposal cli

* chore: fix for init timing

* chore: fix query cli

* chore: fix to prevent generating swap more than MaxSwaps

* fix: add missing call validateBasic for proposal

* fix:

* chore: change error type for temporal test-pass

* chore: fix for make-swap-proposal update

* chore: fix to check swap count after increase in order not to make more than limit

* chore: prevent any proposal with different denom

* chore: change amount cap type

* chore: fix codec-related parts

* chore: minor change for convert to-coin-amount-cap

---------

Signed-off-by: 170210 <j170210@icloud.com>
Co-authored-by: jaeseung.bae <jaeseung.bae@linecorp.com>

---------

Signed-off-by: 170210 <j170210@icloud.com>
Co-authored-by: 170210 <85928898+170210@users.noreply.github.com>
Co-authored-by: Jayden Lee <41176085+tkxkd0159@users.noreply.github.com>
  • Loading branch information
3 people committed May 7, 2024
1 parent 0dd215b commit 5088d1c
Show file tree
Hide file tree
Showing 78 changed files with 28,212 additions and 539 deletions.
10 changes: 8 additions & 2 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,12 @@ Ref: https://keepachangelog.com/en/1.0.0/
## [Unreleased](https://github.com/Finschia/finschia-sdk/compare/v0.48.1...HEAD)

### Features
* (x/fswap) [\#1336](https://github.com/Finschia/finschia-sdk/pull/1336) Initialize fswap module
* (x/fswap) [\#1339](https://github.com/Finschia/finschia-sdk/pull/1339) Implement fswap module's genesis
* (x/fbridge) [\#1340](https://github.com/Finschia/finschia-sdk/pull/1340) Initialize fbridge module
* (x/fbridge) [\#1347](https://github.com/Finschia/finschia-sdk/pull/1347) Implement bridge transfer feature (sending side)
* (x/fbridge) [\#1351](https://github.com/Finschia/finschia-sdk/pull/1351) Map a sequence to block number for every bridge request (sending side)
* (x/fswap) [\#1345](https://github.com/Finschia/finschia-sdk/pull/1345) Implement fswap's basic functionality(MsgSwap, MsgSwapAll, Query, Proposal)

### Improvements
* (types) [\#1314](https://github.com/Finschia/finschia-sdk/pull/1314) replace IsEqual with Equal
Expand All @@ -50,13 +56,13 @@ Ref: https://keepachangelog.com/en/1.0.0/
* (x/collection) [\#1290](https://github.com/Finschia/finschia-sdk/pull/1290) export x/collection params into genesis (backport #1268)
* (x/foundation) [\#1295](https://github.com/Finschia/finschia-sdk/pull/1295) add missing error handling for migration
* (sec) [\#1302](https://github.com/Finschia/finschia-sdk/pull/1302) remove map iteration non-determinism with keys + sorting
* (client) [\#1303](https://github.com/Finschia/finschia-sdk/pull/1303) fix possible overflow in BuildUnsignedTx
* (client) [\#1303](https://github.com/Finschia/finschia-sdk/pull/1303) fix possible overflow in BuildUnsignedTx
* (types) [\#1299](https://github.com/Finschia/finschia-sdk/pull/1299) add missing nil checks
* (x/staking) [\#1301](https://github.com/Finschia/finschia-sdk/pull/1301) Use bytes instead of string comparison in delete validator queue (backport cosmos/cosmos-sdk#12303)
* (client/keys) [#1312](https://github.com/Finschia/finschia-sdk/pull/1312) ignore error when key not found in `keys delete`
* (store) [\#1310](https://github.com/Finschia/finschia-sdk/pull/1310) fix app-hash mismatch if upgrade migration commit is interrupted(backport cosmos/cosmos-sdk#13530)
* (types) [\#1313](https://github.com/Finschia/finschia-sdk/pull/1313) fix correctly coalesce coins even with repeated denominations(backport cosmos/cosmos-sdk#13265)
* (x/crypto) [\#1316](https://github.com/Finschia/finschia-sdk/pull/1316) error if incorrect ledger public key (backport cosmos/cosmos-sdk#14460, cosmos/cosmos-sdk#19691)
* (x/crypto) [\#1316](https://github.com/Finschia/finschia-sdk/pull/1316) error if incorrect ledger public key (backport cosmos/cosmos-sdk#14460, cosmos/cosmos-sdk#19691)
* (x/auth) [#1319](https://github.com/Finschia/finschia-sdk/pull/1319) prevent signing from wrong key in multisig
* (x/mint, x/slashing) [\#1323](https://github.com/Finschia/finschia-sdk/pull/1323) add missing nil check for params validation
* (x/server) [\#1337](https://github.com/Finschia/finschia-sdk/pull/1337) fix panic when defining minimum gas config as `100stake;100uatom`. Use a `,` delimiter instead of `;`. Fixes the server config getter to use the correct delimiter (backport cosmos/cosmos-sdk#18537)
Expand Down
2 changes: 2 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -146,6 +146,8 @@ mocks: $(MOCKS_DIR)
mockgen -package mocks -destination tests/mocks/grpc_server.go github.com/gogo/protobuf/grpc Server
mockgen -package mocks -destination tests/mocks/tendermint_tendermint_libs_log_DB.go github.com/Finschia/ostracon/libs/log Logger
mockgen -source=x/stakingplus/expected_keepers.go -package testutil -destination x/stakingplus/testutil/expected_keepers_mocks.go
mockgen -source=x/fswap/types/expected_keepers.go -package testutil -destination x/fswap/testutil/expected_keepers_mocks.go
mockgen -source=x/fbridge/types/expected_keepers.go -package testutil -destination x/fbridge/testutil/expected_keepers_mocks.go
.PHONY: mocks

$(MOCKS_DIR):
Expand Down
14 changes: 14 additions & 0 deletions client/docs/config.json
Original file line number Diff line number Diff line change
Expand Up @@ -190,6 +190,20 @@
"GetBlockWithTxs": "GetBlockWithTxs2"
}
}
},
{
"url": "./tmp-swagger-gen/lbm/fbridge/v1/query.swagger.json",
"operationIds": {
"rename": {
"Params": "FBridgeParams",
"Commitments": "FBridgeCommitments",
"Guardians": "FBridgeGuardians",
"Operators": "FBridgeOperators",
"Judges": "FBridgeJudges",
"Proposals": "FBridgeProposals",
"Proposal": "FBridgeProposal"
}
}
}
]
}
Loading

0 comments on commit 5088d1c

Please sign in to comment.