Skip to content

Commit

Permalink
docs: update note on unwrapping of account keeper
Browse files Browse the repository at this point in the history
  • Loading branch information
JimLarson authored and mhofman committed Feb 7, 2024
1 parent c27e870 commit 0904225
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions golang/cosmos/app/app.go
Original file line number Diff line number Diff line change
Expand Up @@ -605,10 +605,10 @@ func NewAgoricApp(
app.BaseApp.DeliverTx,
encodingConfig.TxConfig,
),
// REVIEWER: using innerAk here instead of app.AccountKeeper
// NOTE: using innerAk here instead of app.AccountKeeper
// since migration method doesn't know how to unwrap the account keeper.
// Needs access to some struct fields.
// Alternative is to add accessor methods to the AccountKeeper interface.
// (Alternative is to add accessor methods to the AccountKeeper interface.)
auth.NewAppModule(appCodec, innerAk, nil),
vesting.NewAppModule(app.AccountKeeper, app.BankKeeper, app.StakingKeeper),
bank.NewAppModule(appCodec, app.BankKeeper, app.AccountKeeper),
Expand Down

0 comments on commit 0904225

Please sign in to comment.