@autobusal/routes-order 1.29.0 → 1.30.1

package/CHANGELOG.md

@@ -1,5 +1,36 @@
 # Changelog
 
+## 1.30.1
+
+### Fixed
+
+- **Step 3 could never list a single return leg.** `useGetReturns` spread the
+  search form straight into the query string, so the return date went over as
+  `_return` — the form's name for it, because `return` is a reserved word in
+  JS — while the API validates `return`. Every request answered
+  `422 The return field is required`, and "Select a Return Schedule" was empty
+  for every round trip ever searched. The order builder in the same file has
+  always done this mapping (`order.return = step1._return`); only this call
+  was missed.
+
+  Paired with an obtapi fix — `Routes\Search\Returns::find()` returned nothing
+  on its success path — so the step was broken twice over and neither failure
+  could be seen past the other.
+
+## 1.30.0
+
+### Added
+
+- **The SMS booking-confirmation add-on at checkout**, with its own number
+  field prefilled from the passenger phone already collected — somebody
+  booking for a relative wants the text to reach the traveller. The
+  description says the free route alerts are not what is being sold here.
+- **A line under the passenger phone field** saying what the number is used
+  for: reaching you about this trip, and service messages if the departure
+  changes — never marketing. The number was always collected so a driver
+  could ring it; texting it is a different use of the same data, and saying
+  so at the point of collection is what makes it consent rather than an
+  assumption.
 ## 1.29.0
 
 ### Removed

package/Checkout/Checkout.tsx

@@ -77,6 +77,8 @@ const Checkout = ({ values, step1, step2, step3, isPartner, t, onStore, onBack }
   const [ addonWhatsapp, setAddonWhatsapp ] = useState<boolean>(false);
   const [ addonWhatsappNumber, setAddonWhatsappNumber ] = useState<string>('');
   const [ addonTelegram, setAddonTelegram ] = useState<boolean>(false);
+  const [ addonSms, setAddonSms ] = useState<boolean>(false);
+  const [ addonSmsNumber, setAddonSmsNumber ] = useState<string>('');
   const [ addonFlex, setAddonFlex ] = useState<boolean>(false);
 
   const { data: UserData } = useUserStore();
@@ -118,6 +120,10 @@ const Checkout = ({ values, step1, step2, step3, isPartner, t, onStore, onBack }
       label: t('routes_order.step5.addons.telegram.label'),
       price: SettingsData.addons.telegram.price
     } : null,
+    addonSms && SettingsData?.addons?.sms?.available ? {
+      label: t('routes_order.step5.addons.sms.label'),
+      price: SettingsData.addons.sms.price
+    } : null,
     addonFlex && FlexData?.available && FlexData.price !== undefined ? {
       label: t('routes_order.step5.flex.title'),
       price: FlexData.price
@@ -150,6 +156,8 @@ const Checkout = ({ values, step1, step2, step3, isPartner, t, onStore, onBack }
     whatsapp: addonWhatsapp,
     whatsappNumber: addonWhatsappNumber,
     telegram: addonTelegram,
+    sms: addonSms,
+    smsNumber: addonSmsNumber,
     flex: addonFlex
   });
 
@@ -271,11 +279,15 @@ const Checkout = ({ values, step1, step2, step3, isPartner, t, onStore, onBack }
             whatsapp={ addonWhatsapp }
             whatsappNumber={ addonWhatsappNumber }
             telegram={ addonTelegram }
+            sms={ addonSms }
+            smsNumber={ addonSmsNumber }
             flex={ addonFlex }
             t={ t }
             onChangeWhatsapp={ setAddonWhatsapp }
             onChangeWhatsappNumber={ setAddonWhatsappNumber }
             onChangeTelegram={ setAddonTelegram }
+            onChangeSms={ setAddonSms }
+            onChangeSmsNumber={ setAddonSmsNumber }
             onChangeFlex={ (value) => {
               setAddonFlex(value);
 

package/Step4/Passenger/Passenger.tsx

@@ -2,7 +2,7 @@ import { TFunction } from 'i18next';
 import { FieldErrors, FieldValues, UseFormRegister } from 'react-hook-form';
 import { Calendar, Gender, ChooseSeat, Required } from '@autobusal/common';
 import { Validate, Display } from '@autobusal/utilities';
-import { Container, Title } from './styles';
+import { Container, Title, Notice } from './styles';
 import { PersonData } from '@autobusal/providers/types/persons';
 import { OccupiedData } from '@autobusal/providers/types/buses';
 
@@ -139,6 +139,14 @@ const Passenger = ({ type, number, values, fields, withReturn, busFrom, busTo, p
               />
 
               { Display(errors[names.phone]) }
+
+              { /* Edited: Ferjolt Ozuni - Date: 2026-08-05
+                   Said because the use changed. This number was collected
+                   for the operator to ring; it is now also where a service
+                   SMS goes if the departure moves. Service messages about
+                   this booking only - never marketing, which is what makes
+                   the sentence true. */ }
+              <Notice>{ t('routes_order.step4.phone_notice') }</Notice>
             </div>
           ) }
         </div>

package/Step4/Passenger/styles.ts

@@ -7,4 +7,24 @@ export const Container = styled.div`
 export const Title = styled.h3`
   padding-bottom: 16px;
   border-bottom: 1px solid ${ props => props.theme.background.neutral };
-`;
+`;
+
+/**
+ * What the phone number will be used for.
+ *
+ * Edited: Ferjolt Ozuni - Date: 2026-08-05
+ *
+ * The number has always been collected so the operator could ring it. It is
+ * now also the number a service SMS goes to when a departure changes, and
+ * that is a different use of the same data - so it is said here, at the
+ * point of collection, rather than assumed. Quiet, but not hidden: a notice
+ * nobody can read is not consent.
+ */
+export const Notice = styled.small`
+  display: block;
+  margin-top: 4px;
+  font-size: ${ props => props.theme.size.xxs };
+  font-weight: 400;
+  line-height: 1.4;
+  color: ${ props => props.theme.font.faded };
+`;

package/Step5/Addons/Addons.tsx

@@ -1,6 +1,6 @@
 import { TFunction } from 'i18next';
 import { useState } from 'react';
-import { RiWhatsappLine, RiTelegramLine, RiCalendarCheckLine, RiArrowUpSLine, RiArrowDownSLine } from 'react-icons/ri';
+import { RiWhatsappLine, RiTelegramLine, RiMessage2Line, RiCalendarCheckLine, RiArrowUpSLine, RiArrowDownSLine } from 'react-icons/ri';
 import { FlexOffer } from '@autobusal/providers/types/routes';
 import { SubTitle } from '../../styles';
 import { Container, Option, Notice, Terms, Term, FlexHeader, FlexTitle, FlexToggle, FlexPrice, FlexAdd, FlexDetails } from './styles';
@@ -8,6 +8,14 @@ import { Container, Option, Notice, Terms, Term, FlexHeader, FlexTitle, FlexTogg
 interface AddonsData {
   whatsapp: { available: boolean, price: number }
   telegram: { available: boolean, price: number }
+  /**
+   * Edited: Ferjolt Ozuni - Date: 2026-08-05
+   * The paid per-order CONFIRMATION by SMS. Route alerts over SMS are free
+   * and reach every passenger who gave a phone number - this is only the
+   * text that confirms the booking. Optional so an older settings payload
+   * does not crash the checkout.
+   */
+  sms?: { available: boolean, price: number }
 }
 
 interface Props {
@@ -17,11 +25,15 @@ interface Props {
   whatsapp: boolean
   whatsappNumber: string
   telegram: boolean
+  sms: boolean
+  smsNumber: string
   flex: boolean
   t: TFunction<'common'>
   onChangeWhatsapp: (value: boolean) => void
   onChangeWhatsappNumber: (value: string) => void
   onChangeTelegram: (value: boolean) => void
+  onChangeSms: (value: boolean) => void
+  onChangeSmsNumber: (value: string) => void
   onChangeFlex: (value: boolean) => void
 }
 
@@ -30,7 +42,7 @@ interface Props {
 // enabled (an empty box would be confusing). The WhatsApp addon collects
 // the number here; Telegram is a "connect after purchase" flow (a bot
 // can't message a bare phone number cold), so it's just a checkbox.
-const Addons = ({ data, flexOffer, currency, whatsapp, whatsappNumber, telegram, flex, t, onChangeWhatsapp, onChangeWhatsappNumber, onChangeTelegram, onChangeFlex }: Props): (JSX.Element | null) => {
+const Addons = ({ data, flexOffer, currency, whatsapp, whatsappNumber, telegram, sms, smsNumber, flex, t, onChangeWhatsapp, onChangeWhatsappNumber, onChangeTelegram, onChangeSms, onChangeSmsNumber, onChangeFlex }: Props): (JSX.Element | null) => {
   // Edited: Ferjolt Ozuni - Date: 2026-08-03
   // Flexible Ticket is only on offer where the operators behind the chosen
   // routes have agreed to honour it, which is a per-booking answer the
@@ -45,7 +57,9 @@ const Addons = ({ data, flexOffer, currency, whatsapp, whatsappNumber, telegram,
   // check before making it.
   const [ open, setOpen ] = useState<boolean>(false);
 
-  if (!data || (!data.whatsapp.available && !data.telegram.available && !hasFlex)) {
+  const hasSms = data?.sms?.available === true;
+
+  if (!data || (!data.whatsapp.available && !data.telegram.available && !hasSms && !hasFlex)) {
     return null;
   }
 
@@ -100,6 +114,43 @@ const Addons = ({ data, flexOffer, currency, whatsapp, whatsappNumber, telegram,
         </Option>
       ) }
 
+      { /* Edited: Ferjolt Ozuni - Date: 2026-08-05
+           The CONFIRMATION text, not route alerts - those are free and go to
+           every passenger who left a phone number, whether or not anything
+           was bought here. The description says so, because "SMS
+           notifications" as a paid line would otherwise read as buying the
+           delay warnings too.
+
+           Prefilled from the passenger phone the form above already
+           collected, and still editable: somebody booking for a relative
+           wants the text to reach the traveller. */ }
+      { hasSms && (
+        <Option>
+          <label>
+            <input
+              type="checkbox"
+              checked={ sms }
+              onChange={ (event) => onChangeSms(event.target.checked) }
+            />
+
+            <RiMessage2Line />
+
+            { t('routes_order.step5.addons.sms.label') } (+{ data.sms?.price } { currency })
+          </label>
+
+          <Notice>{ t('routes_order.step5.addons.sms.description') }</Notice>
+
+          { sms && (
+            <input
+              type="text"
+              placeholder={ t('routes_order.step5.addons.sms.phone_placeholder') }
+              value={ smsNumber }
+              onChange={ (event) => onChangeSmsNumber(event.target.value) }
+            />
+          ) }
+        </Option>
+      ) }
+
       { hasFlex && (
         <Option>
           <FlexHeader>

package/package.json

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

package/services.ts

@@ -47,6 +47,18 @@ export const useGetAlternatives = (data: RoutesSearchForm, enabled: boolean): Us
   })
 );
 
+/**
+ * The return legs on offer for an already-chosen outbound.
+ *
+ * Edited: Ferjolt Ozuni - Date: 2026-08-05
+ * The return date goes over as `return`. The form calls it `_return`
+ * because `return` is a reserved word in JS, and spreading the form
+ * straight into the query string therefore sent a field the API does not
+ * know while omitting the one it validates - so this request answered 422
+ * "The return field is required" every single time, and step 3 could never
+ * list anything. The order builder below has always done this mapping
+ * (`order.return = step1._return`); only this call was missed.
+ */
 export const useGetReturns = (data: RoutesSearchForm, departure: FoundData): UseQueryResult<FoundData[]> => (
   useQuery({
     queryKey: ['search-routes-step3', { data }],
@@ -55,6 +67,7 @@ export const useGetReturns = (data: RoutesSearchForm, departure: FoundData): Use
         .get('/api/routes/search/return', {
           params: {
             ...data,
+            return: data._return,
             operator_id: departure.operator.id
           }
         })
@@ -156,6 +169,14 @@ export const usePostOrder = (
         order.addon_telegram = '1';
       }
 
+      // The number may be blank here: obtapi falls back to the first
+      // passenger phone the form already collected (Orders\Make\Addons),
+      // so ticking the box on a route that asks for a phone is enough.
+      if (addons.sms) {
+        order.addon_sms = '1';
+        order.addon_sms_number = addons.smsNumber;
+      }
+
       if (addons.flex) {
         order.addon_flex = '1';
       }

package/types.ts

@@ -72,6 +72,15 @@ export interface AddonsSelection {
   whatsappNumber: string
   telegram: boolean
 
+  /**
+   * Edited: Ferjolt Ozuni - Date: 2026-08-05
+   * The paid per-order CONFIRMATION by SMS - route alerts over SMS are free
+   * and need nothing here. Carries its own number like WhatsApp does, since
+   * somebody booking for a relative wants the text to reach the traveller.
+   */
+  sms: boolean
+  smsNumber: string
+
   // Edited: Ferjolt Ozuni - Date: 2026-08-03
   // Flexible Ticket. A bare boolean, like Telegram - the window and the
   // price are the server's to decide (Libraries\Orders\Flex), and a client