@12-apps/discounts 1.19.0 → 1.19.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@12-apps/discounts",
-  "version": "1.19.0",
+  "version": "1.19.1",
   "type": "module",
   "sideEffects": [
     "**/e2e/**"

package/src/react/discount-form-validate.ts

@@ -142,6 +142,20 @@ function checkCombo(
   values: DiscountFormValues,
   slots: readonly ComboRequirement[],
   copy: DiscountsWebCopy,
+): Record<string, string> {
+  const errors = checkComboKind(values, slots, copy);
+  // Recorded apart from `errors.kind`, which holds only ONE message: a bad
+  // quantity wins it, and the product line under the builder must still show.
+  if (values.kind === "FREE_UNITS" && slots.every((slot) => slot.menuItemIds.length === 0)) {
+    errors.freeUnitsTargets = copy.form.freeUnitsTargetRequired;
+  }
+  return errors;
+}
+
+function checkComboKind(
+  values: DiscountFormValues,
+  slots: readonly ComboRequirement[],
+  copy: DiscountsWebCopy,
 ): Record<string, string> {
   if (!isComboKind(values.kind)) return {};
   // "Leve 3, pague 2" is one group of products, so it is refused in those words

package/src/react/discount-target-picker.tsx

@@ -1,6 +1,6 @@
 "use client";
 
-import type { JSX } from "react";
+import { useEffect, type JSX } from "react";
 
 import { useFormContext } from "@12-apps/ui/form/total-form";
 
@@ -87,6 +87,14 @@ export function DiscountTargetPicker({
   const { values, errors, setFieldValue } = useFormContext();
   const kind = values.kind ?? "PERCENTAGE";
   const scope = values.scope ?? "ORDER";
+  const staleTargets = errors.freeUnitsTargets !== undefined;
+
+  // `setFieldValue` clears only the key it sets, and the kind toggle sets
+  // `kind`: the product refusal would outlive a switch away (and keep the error
+  // toast up). Setting its own key drops it; the value is never read or sent.
+  useEffect(() => {
+    if (kind !== "FREE_UNITS" && staleTargets) setFieldValue("freeUnitsTargets", "");
+  }, [kind, staleTargets, setFieldValue]);
 
   if (kind === "FREE_UNITS") {
     return (
@@ -96,6 +104,7 @@ export function DiscountTargetPicker({
         copy={copy}
         freeUnits={values.freeUnits ?? ""}
         error={errors.freeUnits}
+        targetRefused={staleTargets}
         onChange={selection.onComboRequirementsChange}
         onFreeUnitsChange={(next) => setFieldValue("freeUnits", next)}
       />

package/src/react/free-units-builder.tsx

@@ -116,6 +116,7 @@ export function FreeUnitsBuilder({
   copy,
   freeUnits,
   error,
+  targetRefused,
   onChange,
   onFreeUnitsChange,
 }: {
@@ -130,6 +131,8 @@ export function FreeUnitsBuilder({
    * this itself, but it would also sit outside the sentence it belongs to.
    */
   error: string | undefined;
+  /** Criar/Salvar refused the product list; the message waits for it. */
+  targetRefused: boolean;
   onChange: (next: ComboRequirement[]) => void;
   onFreeUnitsChange: (next: string) => void;
 }): JSX.Element | null {
@@ -138,8 +141,6 @@ export function FreeUnitsBuilder({
 
   const slot = onlySlot(slots);
   const summary = summaryOf(slot.quantity, Number(freeUnits), copy);
-  const empty = slot.menuItemIds.length === 0;
-
   return (
     <FormControl fullWidth>
       <FormLabel>{copy.combo.freeUnitsTitle}</FormLabel>
@@ -172,7 +173,7 @@ export function FreeUnitsBuilder({
             {error ?? copy.form.wholeNumberOnly}
           </FormMessage>
         )}
-        {empty ? (
+        {slot.menuItemIds.length === 0 && targetRefused ? (
           <FormMessage error dataTestId="free-units-message">
             {copy.form.freeUnitsTargetRequired}
           </FormMessage>