@autobusal/operator-routes 1.5.1 → 1.5.2

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## 1.5.2
+
+### Removed
+
+- **The confirm dialog in front of removing a stop.** It asked the question
+  the withdrawal panel already asks - the panel names the stop, lists
+  everybody still travelling through it and has its own button - and it
+  answered one part wrongly, warning that the fares would have to be entered
+  again. That stopped being true when calibration began reconciling price
+  rows instead of rebuilding them: measured on a ten-stop route, withdrawing
+  a stop keeps fourteen of sixteen fares and drops only the two legs through
+  the stop itself.
+
 ## 1.3.3
 
 ### Fixed

package/Locations/Browse/Actions.tsx

@@ -24,11 +24,29 @@ const Actions = ({ item, loading, t, onMove, onDelete }: Props): JSX.Element =>
     onMove(order);
   };
 
-  const onAskDelete = (): void => {
-    if (window.confirm(t('locations_manage.actions.delete', { ns: 'common' }))) {
-      onDelete();
-    }
-  };
+  /*
+   * Edited: Claude - Date: 2026-08-22
+   *
+   * No window.confirm here any more. Removing a stop opens the withdrawal
+   * panel, which names the stop, lists everybody still travelling through
+   * it, and has its own explicit button - a far better confirmation than a
+   * browser dialog, and the dialog now stood in front of it saying "YOU WILL
+   * NEED TO ADD THE PRICES AGAIN", which stopped being true when
+   * Locations\Calibration started reconciling price rows instead of
+   * rebuilding them. Measured: withdrawing a stop from a ten-stop route kept
+   * fourteen of sixteen fares, losing only the two legs through the stop
+   * itself.
+   *
+   * So it was a second gate that asked the same question twice and got the
+   * answer wrong once. The panel asks it properly.
+   *
+   * `locations_manage.actions.delete` is deliberately still in obtapi's
+   * language files. obtapi deploys BEFORE the frontends, so for the length
+   * of that window the previous bundle is still calling it - and a missing
+   * key renders as the key itself, which is what the dialog showed when it
+   * was removed. It now says something true and is simply read by nobody
+   * once this bundle lands.
+   */
 
   return (
     <ContainerActions>
@@ -80,7 +98,7 @@ const Actions = ({ item, loading, t, onMove, onDelete }: Props): JSX.Element =>
             </>
           }
           noMargin
-          onClick={ onAskDelete }
+          onClick={ onDelete }
         />
       </Group>
     </ContainerActions>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autobusal/operator-routes",
-  "version": "1.5.1",
+  "version": "1.5.2",
   "author": "Ferjolt Ozuni",
   "type": "module",
   "main": "index.ts"