Skip to content
This repository was archived by the owner on Feb 6, 2024. It is now read-only.

Commit 2d00578

Browse files
committed
fix: don't over migrate
1 parent 70688fd commit 2d00578

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

infra/handler/src/DeckGo/Handler.hs

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -556,7 +556,7 @@ usersPutSession uid u = do
556556
1 -> do
557557
HS.sql "COMMIT"
558558
pure UserUpdateOk
559-
0 -> do
559+
_ -> do
560560
HS.sql "ROLLBACK"
561561
pure UserUpdateClash
562562

@@ -1129,7 +1129,12 @@ data DbVersion
11291129

11301130
-- | Migrates from ver to latest
11311131
migrateFrom :: DbVersion -> HS.Session ()
1132-
migrateFrom = \ver -> forM_ [ver .. maxBound] migrateTo
1132+
migrateFrom = \ver ->
1133+
if ver < maxBound
1134+
then
1135+
let from = succ ver
1136+
in forM_ [from .. maxBound] migrateTo
1137+
else pure ()
11331138
where
11341139
-- | Migrates from (ver -1) to ver
11351140
migrateTo :: DbVersion -> HS.Session ()

0 commit comments

Comments
 (0)