Fix bug in airdrop script from claims module removal (#1629)

* Fix bug in airdrop script from claims module removal * Fix bug in the airdrop claim script, introduced from #1301 Co-authored-by: Aleksandr Bezobchuk <[email protected]>
osmosis-labs · May 31, 2022 · 99419b1 · 99419b1
1 parent 8fe02f0
commit 99419b1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -84,6 +84,7 @@ Cosmwasm bindings have been added, to make swapping and creating these new token
 
 ### Other Features
 
+* [#1629](https://github.com/osmosis-labs/osmosis/pull/1629) Fix bug in the airdrop claim script
 * [#1570](https://github.com/osmosis-labs/osmosis/pull/1570) upgrade sdk with app version fix for state-sync
 * [#1554](https://github.com/osmosis-labs/osmosis/pull/1554) local dev environment
 * [#1541](https://github.com/osmosis-labs/osmosis/pull/1541) Add arm64 support to Docker

diff --git a/cmd/osmosisd/cmd/balances_from_state_export.go b/cmd/osmosisd/cmd/balances_from_state_export.go
@@ -270,7 +270,7 @@ Example:
 			}
 
 			// convert balances to underlying coins and sum up balances to total balance
-			for _, account := range snapshotAccs {
+			for addr, account := range snapshotAccs {
 				// All pool shares are in liquid balances OR bonded balances (locked),
 				// therefore underlyingCoinsForSelectPools on liquidBalances + bondedBalances
 				// will include everything that is in one of those two pools.
@@ -283,6 +283,7 @@ Example:
 					Add(sdk.NewCoin(appparams.BaseCoinUnit, account.Staked)).
 					Add(sdk.NewCoin(appparams.BaseCoinUnit, account.UnbondingStake)).
 					Add(account.Bonded...)
+				snapshotAccs[addr] = account
 			}
 
 			snapshot := DeriveSnapshot{