@arquimedes.co/eureka-forms 3.0.69 → 3.0.70

package/dist/FormSteps/BookingStep/MaterialBookingStep/MaterialBookingStep.js

@@ -124,6 +124,10 @@ function MaterialBookingStep({ step, editable, fetchSlots: customFetchSlots }) {
     const personSelectionMode = usesEdge
         ? (offer?.personSelection ?? PersonSelectionMode.NONE)
         : step.personSelectionMode;
-    return (_jsx("div", { ref: ref, style: cssVars, children: _jsx(ErkBookingPicker, { value: value, onChange: handleChange, label: step.label, description: step.description, durationMinutes: offer?.durationMinutes ?? step.durationMinutes, bufferMinutes: step.bufferMinutes, bookingMode: step.bookingMode, temporaryLockMinutes: step.temporaryLockMinutes, personSelectionMode: personSelectionMode, persons: usesEdge ? persons : step.persons, collectContact: step.collectContact, fetchSlots: usesEdge ? fetchSlots : customFetchSlots, fetchDays: usesEdge ? fetchDays : undefined, holdSlot: usesEdge ? holdSlot : undefined, disabled: usesEdge && !offer, readOnly: isReadOnly, error: error?.message ?? offerError }) }));
+    return (_jsx("div", { ref: ref, style: cssVars, children: _jsx(ErkBookingPicker, { value: value, onChange: handleChange, label: step.label, description: step.description, durationMinutes: offer?.durationMinutes ?? step.durationMinutes, bufferMinutes: step.bufferMinutes, bookingMode: step.bookingMode, temporaryLockMinutes: step.temporaryLockMinutes, personSelectionMode: personSelectionMode, persons: usesEdge ? persons : step.persons, 
+            // Whoever holds a slot must be named unless the step lets strangers book; asking for the name
+            // whenever it is required keeps the hold from being refused for a step that never turned the
+            // contact fields on.
+            collectContact: !step.allowAnonymous || step.collectContact === true, requireContact: !step.allowAnonymous, fetchSlots: usesEdge ? fetchSlots : customFetchSlots, fetchDays: usesEdge ? fetchDays : undefined, holdSlot: usesEdge ? holdSlot : undefined, disabled: usesEdge && !offer, readOnly: isReadOnly, error: error?.message ?? offerError }) }));
 }
 export default MaterialBookingStep;

package/dist/Shared/ErkBookingPicker/ErkBookingPicker.d.ts

@@ -12,6 +12,8 @@ export interface ErkBookingPickerProps {
     persons?: BookablePerson[];
     /** Recopila nombre/correo del que agenda. */
     collectContact?: boolean;
+    /** El nombre es obligatorio para confirmar: el paso no admite reservas anónimas. */
+    requireContact?: boolean;
     /** Provee los slots disponibles para una persona + día. */
     fetchSlots?: (args: {
         personId?: string;
@@ -48,5 +50,5 @@ export interface ErkBookingPickerProps {
  * En el paso final el panel derecho desaparece y el resumen se expande en una columna
  * centrada (cross-fade corto). Confirmar cierra el modal y deja el resumen compacto.
  */
-declare function ErkBookingPicker({ value, onChange, label, description, durationMinutes, bufferMinutes, bookingMode, temporaryLockMinutes, personSelectionMode, persons, collectContact, fetchSlots, fetchDays, holdSlot, disabled, readOnly, error, }: ErkBookingPickerProps): JSX.Element;
+declare function ErkBookingPicker({ value, onChange, label, description, durationMinutes, bufferMinutes, bookingMode, temporaryLockMinutes, personSelectionMode, persons, collectContact, requireContact, fetchSlots, fetchDays, holdSlot, disabled, readOnly, error, }: ErkBookingPickerProps): JSX.Element;
 export default ErkBookingPicker;

package/dist/Shared/ErkBookingPicker/ErkBookingPicker.js

@@ -59,7 +59,7 @@ function capitalize(text) {
  * En el paso final el panel derecho desaparece y el resumen se expande en una columna
  * centrada (cross-fade corto). Confirmar cierra el modal y deja el resumen compacto.
  */
-function ErkBookingPicker({ value, onChange, label, description, durationMinutes, bufferMinutes, bookingMode, temporaryLockMinutes, personSelectionMode, persons, collectContact = false, fetchSlots, fetchDays, holdSlot, disabled = false, readOnly = false, error, }) {
+function ErkBookingPicker({ value, onChange, label, description, durationMinutes, bufferMinutes, bookingMode, temporaryLockMinutes, personSelectionMode, persons, collectContact = false, requireContact = false, fetchSlots, fetchDays, holdSlot, disabled = false, readOnly = false, error, }) {
     const needsPerson = !!personSelectionMode && personSelectionMode !== PersonSelectionMode.NONE;
     const personsOptional = personSelectionMode === PersonSelectionMode.OPTIONAL;
     const today = startOfDay(new Date());
@@ -160,6 +160,8 @@ function ErkBookingPicker({ value, onChange, label, description, durationMinutes
                 return !!date;
             case 'hour':
                 return !!slot;
+            case 'confirm':
+                return !requireContact || contactName.trim().length > 0;
             default:
                 return true;
         }
@@ -287,6 +289,6 @@ function ErkBookingPicker({ value, onChange, label, description, durationMinutes
                                                         ? 'Si todo está bien, completa tus datos y confirma.'
                                                         : 'Si todo está bien, confirma tu cita.' })] }, "confirm"))] })] }), _jsxs("div", { className: styles.navbar, children: [_jsx("div", { className: styles.navSummary, children: isFinal || !canAdvance || navParts.length === 0
                                         ? STEP_HINT[currentKey]
-                                        : navParts.reduce((acc, part, i) => i === 0 ? [part] : [...acc, _jsx("span", { children: " \u00B7 " }, `s${i}`), part], []) }), wizStep > 0 && (_jsx(ErkButton, { text: "Atr\u00E1s", variant: "outlined", onClick: () => goToStep(wizStep - 1) })), isFinal ? (_jsx(ErkButton, { text: holding ? 'Reservando…' : 'Confirmar cita', sx: ACCENT_SX, disabled: !slot || holding, onClick: () => void confirm() })) : (_jsx(ErkButton, { text: "Siguiente", disabled: !canAdvance, onClick: () => goToStep(wizStep + 1) }))] })] }) }), error && _jsx("div", { className: styles.err, children: error })] }));
+                                        : navParts.reduce((acc, part, i) => i === 0 ? [part] : [...acc, _jsx("span", { children: " \u00B7 " }, `s${i}`), part], []) }), wizStep > 0 && (_jsx(ErkButton, { text: "Atr\u00E1s", variant: "outlined", onClick: () => goToStep(wizStep - 1) })), isFinal ? (_jsx(ErkButton, { text: holding ? 'Reservando…' : 'Confirmar cita', sx: ACCENT_SX, disabled: !slot || holding || !canAdvance, onClick: () => void confirm() })) : (_jsx(ErkButton, { text: "Siguiente", disabled: !canAdvance, onClick: () => goToStep(wizStep + 1) }))] })] }) }), error && _jsx("div", { className: styles.err, children: error })] }));
 }
 export default ErkBookingPicker;

package/dist/Shared/ErkBookingPicker/ErkBookingPicker.test.js

@@ -141,6 +141,20 @@ describe('ErkBookingPicker (wizard)', () => {
         expect(fetchSlots).toHaveBeenCalledTimes(2);
         expect(onChange).not.toHaveBeenCalled();
     });
+    it('does not confirm without a name when the step requires one', async () => {
+        const user = userEvent.setup();
+        const onChange = vi.fn();
+        render(_jsx(MaterialProviders, { formStyle: InternalFormStyle, children: _jsx(ErkBookingPicker, { value: null, onChange: onChange, label: "Agendar cita", durationMinutes: 30, personSelectionMode: PersonSelectionMode.NONE, collectContact: true, requireContact: true, fetchSlots: () => Promise.resolve(SLOTS) }) }));
+        await user.click(screen.getByRole('button', { name: 'Agendar cita' }));
+        await user.click(clickableToday());
+        await user.click(screen.getByRole('button', { name: 'Siguiente' }));
+        await user.click(await screen.findByRole('button', { name: '09:00' }));
+        await user.click(screen.getByRole('button', { name: 'Siguiente' }));
+        const confirm = await screen.findByRole('button', { name: 'Confirmar cita' });
+        expect(confirm).toBeDisabled();
+        await user.type(screen.getByLabelText('Nombre'), 'Juan');
+        expect(confirm).toBeEnabled();
+    });
     it('renders the compact summary when a value exists', () => {
         render(_jsx(MaterialProviders, { formStyle: InternalFormStyle, children: _jsx(ErkBookingPicker, { value: {
                     type: 'BOOKING_VALUE',

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@arquimedes.co/eureka-forms",
   "repository": "git://github.com/Arquimede5/Eureka-Forms.git",
-  "version": "3.0.69",
+  "version": "3.0.70",
   "scripts": {
     "watch": "node node_modules/@typescript/native/bin/tsc --noEmit --watch --project tsconfig.app.json",
     "start": "vite",