jpbeldomenico/taller_automotriz
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: bot de reservas por WhatsApp (F0) y flujo web jefe -> trabajo #4

Merged
jpbeldomenico merged 6 commits from feat/whatsapp-bot into main 2026-08-05 12:24:55 -03:00
Conversation 87 Commits 6 Files Changed 23
+2 -2
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 2df679777c - Show all commits
src/app/api/jefe/servicios/[id]/route.ts
+1 -1
View File
@@ -20,7 +20,7 @@ export async function PATCH(req: NextRequest, { params }: { params: Promise<{ id
const dur = Number(data.duracionHoras);
if (!Number.isFinite(dur) || dur <= 0 || dur > 24)
ai-bot commented 2026-08-05 12:05:29 -03:00
Review
Copy Link
Copy Source

Duration validation rejects 0 but error message says 'entre 0 y 24'. Should be 'entre 1 y 24' or allow 0 with dur < 0.

if (!Number.isFinite(dur) || dur < 0 || dur > 24)

#ai-review-inline

Duration validation rejects 0 but error message says 'entre 0 y 24'. Should be 'entre 1 y 24' or allow 0 with `dur < 0`. ```suggestion if (!Number.isFinite(dur) || dur < 0 || dur > 24) ``` #ai-review-inline
return NextResponse.json(
{ error: "La duración debe ser un número de horas entre 0 y 24" },
{ error: "La duración debe ser mayor que 0 y hasta 24 horas" },
ai-bot commented 2026-08-05 12:05:29 -03:00
Outdated
Review
Copy Link
Copy Source

Error message inconsistent with validation logic (allows 0 vs rejects 0).

{ error: "La duración debe ser un número de horas entre 1 y 24" }

#ai-review-inline

Error message inconsistent with validation logic (allows 0 vs rejects 0). ```suggestion { error: "La duración debe ser un número de horas entre 1 y 24" } ``` #ai-review-inline
{ status: 400 }
);
}
2
src/app/api/jefe/servicios/route.ts
+1 -1
View File
2
 @@ -28,7 +28,7 @@ export async function POST(req: NextRequest) {
return NextResponse.json({ error: "El nombre del trabajo es obligatorio" }, { status: 400 });
if (!Number.isFinite(dur) || dur <= 0 || dur > 24)
ai-bot commented 2026-08-05 12:06:42 -03:00
Review
Copy Link
Copy Source

Error message says 'entre 0 y 24' but validation rejects 0 (dur <= 0). Update message to 'entre 1 y 24' or adjust validation to allow 0.

      { error: "La duración debe ser un número de horas entre 1 y 24" },

#ai-review-inline

Error message says 'entre 0 y 24' but validation rejects 0 (dur <= 0). Update message to 'entre 1 y 24' or adjust validation to allow 0. ```suggestion { error: "La duración debe ser un número de horas entre 1 y 24" }, ``` #ai-review-inline
ai-bot commented 2026-08-05 12:06:43 -03:00
Review
Copy Link
Copy Source

Magic number 24 for max hours. Define as constant for maintainability.

  const MAX_HOURS = 24;
  if (!Number.isFinite(dur) || dur <= 0 || dur > MAX_HOURS)

#ai-review-inline

Magic number 24 for max hours. Define as constant for maintainability. ```suggestion const MAX_HOURS = 24; if (!Number.isFinite(dur) || dur <= 0 || dur > MAX_HOURS) ``` #ai-review-inline
return NextResponse.json(
{ error: "La duración debe ser un número de horas entre 0 y 24" },
{ error: "La duración debe ser mayor que 0 y hasta 24 horas" },
{ status: 400 }
);
1