changelog, add detail err log

cosmos · Jun 11, 2024 · eae8154 · eae8154
1 parent 053ae26
commit eae8154
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -199,6 +199,7 @@ Every module contains its own CHANGELOG.md. Please refer to the module you are i
 
 * (simapp) [#19146](https://github.com/cosmos/cosmos-sdk/pull/19146) Replace `--v` CLI option with `--validator-count`/`-n`.
 * (module) [#19370](https://github.com/cosmos/cosmos-sdk/pull/19370) Deprecate `module.Configurator`, use `appmodule.HasMigrations` and `appmodule.HasServices` instead from Core API.
+* (x/auth) [#20531](https://github.com/cosmos/cosmos-sdk/pull/20531) Deprecate auth keeper `NextAccountNumber`, use `keeper.AccountsModKeeper.NextAccountNumber` instead.
 
 ## [v0.50.7](https://github.com/cosmos/cosmos-sdk/releases/tag/v0.50.7) - 2024-06-04
 

diff --git a/x/auth/keeper/keeper.go b/x/auth/keeper/keeper.go
@@ -342,11 +342,11 @@ func (ak AccountKeeper) NonAtomicMsgsExec(ctx context.Context, signer sdk.AccAdd
 // MigrateAccountNumberUnsafe migrates auth's account number to accounts's account number
 // and delete store entry for auth legacy GlobalAccountNumberKey.
 //
-// Should only use in an upgrade handler for migrate process.
+// Should only use in an upgrade handler for migrating account number.
 func (ak AccountKeeper) MigrateAccountNumberUnsafe(ctx context.Context) error {
 	currentAccNum, err := ak.RemoveLegacyAccountNumber(ctx)
 	if err != nil {
-		return err
+		return fmt.Errorf("failed to migrate account number: %w", err)
 	}
 
 	err = ak.AccountsModKeeper.InitAccountNumberSeqUnsafe(ctx, currentAccNum)