fix: activity value should be positive

synonymdev · Aug 1, 2023 · c291d4b · c291d4b
1 parent d43d4dc
commit c291d4b
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/src/screens/Activity/ActivityDetail.tsx b/src/screens/Activity/ActivityDetail.tsx
@@ -694,7 +694,7 @@ const LightningActivityDetail = ({
 	return (
 		<>
 			<Money
-				sats={value}
+				sats={total}
 				unitType="secondary"
 				size="caption13Up"
 				color="gray1"

diff --git a/src/store/index.ts b/src/store/index.ts
@@ -37,7 +37,7 @@ const persistConfig = {
 	key: 'root',
 	storage: mmkvStorage,
 	// increase version after store shape changes
-	version: 14,
+	version: 15,
 	stateReconciler: autoMergeLevel2,
 	blacklist: ['ui'],
 	migrate: createMigrate(migrations, { debug: __ENABLE_MIGRATION_DEBUG__ }),

diff --git a/src/store/migrations/index.ts b/src/store/migrations/index.ts
@@ -141,15 +141,26 @@ const migrations = {
 		};
 	},
 	15: (state): PersistedState => {
+		// remove old unit settings
 		delete state.settings.bitcoinUnit;
 		delete state.settings.balanceUnit;
 
+		// LN activity value should be positive
+		const items = state.activity.items.map((item: { value: number }) => ({
+			...item,
+			value: Math.abs(item.value),
+		}));
+
 		return {
 			...state,
 			settings: {
 				...state.settings,
 				unit: 'satoshi',
 			},
+			activity: {
+				...state.activity,
+				items,
+			},
 		};
 	},
 };