Merge pull request #84 from b-harvest/patch/upgrade-handler

fix: v8 upgrade handler

app/upgrades/v8/upgrades.go

@@ -31,10 +31,15 @@ func CreateUpgradeHandler(
 		sdkCtx := sdk.UnwrapSDKContext(ctx)
 		logger := sdkCtx.Logger().With("upgrade: ", UpgradeName)
 
+		var (
+			updatedVM module.VersionMap
+			err       error
+		)
+
 		// Leave modules are as-is to avoid running InitGenesis.
 		logger.Debug("running module migrations ...")
-		if vm, err := mm.RunMigrations(ctx, configurator, vm); err != nil {
-			return vm, err
+		if updatedVM, err = mm.RunMigrations(ctx, configurator, vm); err != nil {
+			return updatedVM, err
 		}
 
 		// ibc-go vX -> v6
@@ -55,19 +60,19 @@ func CreateUpgradeHandler(
 		}
 
 		if err := baseapp.MigrateParams(sdkCtx, legacySubspace, consensusParamsStore); err != nil {
-			return vm, err
+			return updatedVM, err
 		}
 
 		// canto v8 custom
 		{
 			params, err := stakingKeeper.GetParams(ctx)
 			if err != nil {
-				return vm, err
+				return updatedVM, err
 			}
 			params.MinCommissionRate = MinCommissionRate
 			stakingKeeper.SetParams(ctx, params)
 		}
 
-		return vm, nil
+		return updatedVM, nil
 	}
 }