@autobusal/routes-order 1.33.2 → 1.33.4

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 1.33.4
+
+### Added
+
+- **Complimentary in the web checkout.** An operator/employee selling on their OWN operator's route is offered the zero-cost Complimentary option the mobile app already had - the route-aware flag comes from `/api/funds/allowed?route=...` and the ownership rule is re-checked server-side at payment time. (Audit decision BOOK-07; needs obtapi >= 2.4.37.)
+
+## 1.33.3
+
+### Fixed
+
+- **Checkout swallowed every booking error.** A 4xx from order creation - a seat taken while the form was open, the unpaid-hold limit, a route gone unavailable - left the buyer on the filled form with no message. It now surfaces the API's message. (Audit BOOK-13-c.)
+
 ## 1.33.2
 
 ### Fixed

package/Checkout/Checkout.tsx

@@ -27,6 +27,7 @@ import { useGetSettings } from '@autobusal/providers/services';
 import { usePostOrder, useGetFlex, useGetPayments, useGetSavedPassengers } from '../services';
 import { addon as commerceAddon, beginCheckout, stash } from '@autobusal/providers/Setup/commerce';
 import { item as commerceItem } from '../commerce';
+import { error as notifyError } from '@autobusal/utilities';
 
 interface Props {
   values?: PersonData
@@ -113,7 +114,7 @@ const Checkout = ({ values, step1, step2, step3, isPartner, t, onStore, onBack }
    * answers it from cache - this reads the answer, it does not fetch it a
    * second time.
    */
-  const { data: paymentsData } = useGetPayments();
+  const { data: paymentsData } = useGetPayments(step2.id);
 
   const singleMethod = getAvailablePayments(paymentsData, isPartner ?? false).length === 1;
 
@@ -351,6 +352,20 @@ const Checkout = ({ values, step1, step2, step3, isPartner, t, onStore, onBack }
         );
 
         navigate(`/orders/process/${ action }/${ hash }`);
+      },
+      /*
+       * Edited: Claude - Date: 2026-08-22 (audit finding BOOK-13-c)
+       *
+       * Surface why the booking was refused. Every 4xx from order creation -
+       * a seat taken while the form was open, the unpaid-hold limit, a route
+       * that just went unavailable - left the buyer on the filled form with
+       * no message, clicking Pay again. The API's own message says exactly
+       * what happened.
+       */
+      onError: (submitError: unknown) => {
+        const message = (submitError as { response?: { data?: { message?: string } } })?.response?.data?.message;
+
+        notifyError(message ?? t('routes_order.checkout.error', { ns: 'common', defaultValue: 'We could not complete your booking. Please try again.' }));
       }
     });
   };
@@ -432,6 +447,7 @@ const Checkout = ({ values, step1, step2, step3, isPartner, t, onStore, onBack }
             <Payments
               selected={ action ?? '' }
               isPartner={ isPartner ?? false }
+              routeId={ step2.id }
               t={ t }
               onChange={ onAction }
             />

package/Step5/Payments/Payments.tsx

@@ -1,7 +1,7 @@
 import { useEffect } from 'react';
 import { TFunction } from 'i18next';
 import { RiSecurePaymentLine } from 'react-icons/ri';
-import { FaRegHandPaper, FaMoneyBill } from 'react-icons/fa';
+import { FaRegHandPaper, FaMoneyBill, FaGift } from 'react-icons/fa';
 import { AiOutlineCreditCard } from 'react-icons/ai';
 import { Inline } from '@autobusal/common';
 import { SubTitle } from '../../styles';
@@ -12,12 +12,13 @@ import { getAvailablePayments } from '../utilities';
 interface Props {
   selected: string
   isPartner: boolean
+  routeId?: number | string
   t: TFunction<'common'>
   onChange: (action: string) => void
 }
 
-const Payments = ({ selected, isPartner, t, onChange }: Props): JSX.Element | null => {
-  const { data, isLoading, isSuccess } = useGetPayments();
+const Payments = ({ selected, isPartner, routeId, t, onChange }: Props): JSX.Element | null => {
+  const { data, isLoading, isSuccess } = useGetPayments(routeId);
 
   const available = getAvailablePayments(data, isPartner);
 
@@ -90,6 +91,13 @@ const Payments = ({ selected, isPartner, t, onChange }: Props): JSX.Element | nu
             </ButtonPayment>
           ) }
 
+          { (data?.complimentary && !isPartner) && (
+            <ButtonPayment type="button" $selected={ selected === 'complimentary' } onClick={ () => onChange('complimentary') }>
+              <FaGift />
+              { t('routes_order.step5.actions.complimentary') }
+            </ButtonPayment>
+          ) }
+
           { (data?.bkt || isPartner) && (
             <ButtonPayment type="button" $selected={ selected === 'bkt' } onClick={ () => onChange('bkt') }>
               <AiOutlineCreditCard />

package/Step5/utilities.ts

@@ -51,6 +51,11 @@ export const getAvailablePayments = (data?: ActionData, isPartner = false): stri
   if (data.reserve && !isPartner) { available.push('reserve'); }
   if (data.funds && !isPartner) { available.push('funds', 'cash'); }
 
+  // Claude - 2026-08-22 (audit decision BOOK-07): an operator/employee on
+  // their own operator's route pays nothing - the flag is route-aware
+  // (services.ts) and re-checked server-side at payment time.
+  if (data.complimentary && !isPartner) { available.push('complimentary'); }
+
   if (data.bkt || isPartner) { available.push('bkt'); }
   if (data.stripe || isPartner) { available.push('stripe'); }
   if (data.mollie || isPartner) { available.push('mollie'); }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autobusal/routes-order",
-  "version": "1.33.2",
+  "version": "1.33.4",
   "author": "Ferjolt Ozuni",
   "type": "module",
   "main": "index.ts"

package/services.ts

@@ -222,12 +222,20 @@ export const useGetCoupon = (routeId: number, total: number): UseMutationResult<
   })
 );
 
-export const useGetPayments = (): UseQueryResult<ActionData> => (
+/*
+ * Claude - 2026-08-22 (audit decision BOOK-07): asks with the route being
+ * sold, so the API can offer Complimentary to an operator/employee on
+ * their own operator's route. External routes carry string ids and can
+ * never be own inventory, so those simply don't send the parameter.
+ */
+export const useGetPayments = (routeId?: number | string): UseQueryResult<ActionData> => (
   useQuery({
-    queryKey: ['allowed-funds'],
+    queryKey: ['allowed-funds', { routeId }],
     queryFn: async () => (
       await apiClient
-        .get('/api/funds/allowed')
+        .get('/api/funds/allowed', {
+          params: typeof routeId === 'number' ? { route: routeId } : {}
+        })
         .then(response => (
           response.data
         ))

package/types.ts

@@ -42,6 +42,7 @@ export interface ActionData {
   fake: boolean
   reserve: boolean
   funds: boolean
+  complimentary?: boolean
   bkt: boolean
   stripe: boolean
   mollie: boolean