From 95a368163a3210b1004dc92ba552630dd6605bdc Mon Sep 17 00:00:00 2001
From: Shawn <shawn@ourpad.casa>
Date: Tue, 2 Jun 2026 18:50:57 -0400
Subject: [PATCH] fix: remove duplicate doAutoCheckin call from handleBarcode

showScannedAsset (called on the very next line) already triggers the
auto-checkin. handleBarcode was calling doAutoCheckin immediately before
showScannedAsset, resulting in two check-in records per barcode scan.
---
 static/index.html | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/static/index.html b/static/index.html
index b0a2d9a..c9c8b0e 100644
--- a/static/index.html
+++ b/static/index.html
@@ -3108,10 +3108,6 @@
           const asset = await api(`/api/assets/search?machine_id=${encodeURIComponent(machineId)}`);
           // Asset exists — show result in-place with scan new button
           showToast(`${esc(asset.name)} — found!`, false);
-          // Auto-checkin if GPS available (background, non-blocking)
-          if (AppState.gpsLat != null) {
-            doAutoCheckin(asset.id);
-          }
           showScannedAsset(asset);
         } catch (e) {
           if (e.message === 'Asset not found' || e.message.includes('404')) {