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.

Sign up for GitHub

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

feat: Allow to restrict MintCoins from app.go #10771

Merged
merged 13 commits into from
Feb 18, 2022
Merged

feat: Allow to restrict MintCoins from app.go #10771

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
153274c
Feat: set min-coins-restriction for bank module
mattverse Dec 14, 2021
7a345b3
Add nesting restriction fns, Add type alias
mattverse Dec 15, 2021
39c8037
Change to table driven tests
mattverse Dec 15, 2021
5f2fc6b
Add changelog entry and excerpt to spec
mattverse Jan 4, 2022
f41cd99
Update documentation for minting perms
mattverse Jan 4, 2022
2014b9e
Merge branch 'master' into mattverse/restrict-mint-coins
ValarDragon Jan 5, 2022
f7ddd0a
Merge branch 'master' into mattverse/restrict-mint-coins
ValarDragon Jan 14, 2022
95b6fdd
Merge branch 'master' into mattverse/restrict-mint-coins
ValarDragon Jan 19, 2022
14140be
Merge branch 'master' into mattverse/restrict-mint-coins
ValarDragon Jan 19, 2022
fa2b7ee
Update go doc and comments
mattverse Jan 26, 2022
77e5c4d
Modify CHANGELOG.md
mattverse Jan 28, 2022
8fd7bd9
Merge branch 'master' into mattverse/restrict-mint-coins
mattverse Jan 28, 2022
7dade4b
Merge branch 'master' into mattverse/restrict-mint-coins
tac0turtle Feb 18, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
* (types) [\#10630](https://github.com/cosmos/cosmos-sdk/pull/10630) Add an `Events` field to the `TxResponse` type that captures _all_ events emitted by a transaction, unlike `Logs` which only contains events emitted during message execution.
* (deps) [\#10706](https://github.com/cosmos/cosmos-sdk/issues/10706) Bump rosetta-sdk-go to v0.7.2 and rosetta-cli to v0.7.3
* (module) [\#10711](https://github.com/cosmos/cosmos-sdk/pull/10711) Panic at startup if the app developer forgot to add modules in the `SetOrder{BeginBlocker, EndBlocker, InitGenesis, ExportGenesis}` functions. This means that all modules, even those who have empty implementations for those methods, need to be added to `SetOrder*`.

* (x/bank) [\#10771](https://github.com/cosmos/cosmos-sdk/pull/10771) Add safety check on bnak module perms to only mint specified denom.
mattverse marked this conversation as resolved.
Show resolved Hide resolved
### Bug Fixes

* [\#10414](https://github.com/cosmos/cosmos-sdk/pull/10414) Use `sdk.GetConfig().GetFullBIP44Path()` instead `sdk.FullFundraiserPath` to generate key
Expand Down
1 change: 1 addition & 0 deletions x/bank/spec/02_keepers.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,7 @@ The base keeper provides full-permission access: the ability to arbitrary modify
// between accounts.
type Keeper interface {
SendKeeper
WithMintCoinsRestriction(NewRestrictionFn BankMintingRestrictionFn) BaseKeeper
ValarDragon marked this conversation as resolved.
Show resolved Hide resolved

InitGenesis(sdk.Context, *types.GenesisState)
ExportGenesis(sdk.Context) *types.GenesisState
Expand Down