Fix sales date, responsiveness and automatic closing of the scheduling modal

Author: emanuellcs

actions/appointments.ts

@@ -42,6 +42,28 @@ export async function createAppointmentAction(
       };
     }
 
+    // Sync sale's created_at to appointment's start_time
+    if (data?.id) {
+      try {
+        const appointmentId = parseInt(data.id, 10);
+        const appointmentStartTime = appointment.startTime;
+        
+        await supabase
+          .from("sales")
+          .update({ created_at: appointmentStartTime })
+          .eq("appointment_id", appointmentId);
+        
+        console.log(
+          `Sale created_at synced to appointment start_time: ${appointmentStartTime}`
+        );
+      } catch (syncError) {
+        console.warn(
+          "Failed to sync sale created_at to appointment start_time:",
+          syncError
+        );
+      }
+    }
+
     revalidatePath("/agenda");
     revalidatePath("/financeiro");
 

actions/finance.ts

@@ -27,14 +27,35 @@ export async function createSaleAction(sale: NewSale) {
     // Fetch appointment start time if linked
     let inheritedCreatedAt = sale.createdAt || new Date().toISOString();
     if (sale.appointmentId) {
-      const { data: appt } = await supabase
-        .from("appointments")
-        .select("start_time")
-        .eq("id", parseInt(sale.appointmentId))
-        .single();
-      
-      if (appt?.start_time) {
-        inheritedCreatedAt = appt.start_time;
+      try {
+        const appointmentId = parseInt(sale.appointmentId, 10);
+        if (isNaN(appointmentId)) {
+          console.warn(
+            `Invalid appointmentId provided to createSaleAction: ${sale.appointmentId}`
+          );
+        } else {
+          const { data: appt, error: apptErr } = await supabase
+            .from("appointments")
+            .select("start_time")
+            .eq("id", appointmentId)
+            .single();
+
+          if (apptErr) {
+            console.warn(
+              `Failed to fetch appointment ${appointmentId}: ${apptErr.message}`
+            );
+          } else if (appt?.start_time) {
+            inheritedCreatedAt = appt.start_time;
+            console.log(
+              `Sale linked to appointment ${appointmentId}: using start_time ${appt.start_time}`
+            );
+          }
+        }
+      } catch (appointmentFetchError) {
+        console.error(
+          "Error fetching appointment for sale creation:",
+          appointmentFetchError
+        );
       }
     }
 

app/agenda/page.tsx

@@ -248,20 +248,29 @@ export default function AgendaPage() {
 
         const supabaseRes = await addAppointment(supabasePayload);
         if (supabaseRes) {
+          // Close modal immediately after successful DB save (independent operation)
+          setIsModalOpen(false);
+          toast.success("Agendamento criado!");
+          // Attempt Google Calendar sync (non-blocking)
           res = await createCalendarEvent(googlePayload);
+          if (!res?.success) {
+            toast.error(res?.error || "Erro ao sincronizar com Google Agenda");
+          }
         } else {
           throw new Error("Falha ao salvar no banco de dados.");
         }
       }
 
       if (res?.success) {
-        toast.success(
-          selectedEvent ? "Agendamento atualizado!" : "Agendamento criado!",
-        );
-        setIsModalOpen(false);
+        if (selectedEvent) {
+          // Edit mode: only show success toast for Google Calendar result
+          toast.success("Agendamento atualizado!");
+          setIsModalOpen(false);
+        }
         fetchEvents();
-      } else {
-        toast.error(res?.error || "Erro ao salvar no Google Agenda");
+      } else if (selectedEvent) {
+        // Edit mode: Google Calendar sync failed
+        toast.error(res?.error || "Erro ao atualizar no Google Agenda");
       }
     } catch (e) {
       console.error(e);

app/financeiro/page.tsx

@@ -381,14 +381,14 @@ export default function FinanceiroPage() {
               </Button>
 
               {showFilters && (
-                <div className="flex items-center gap-2 animate-in fade-in slide-in-from-right-1">
+                <div className="flex flex-wrap items-center gap-1 sm:gap-2 animate-in fade-in slide-in-from-right-1">
                   <div className="flex items-center gap-1.5">
                     <Label className="text-[10px] font-black uppercase tracking-widest text-muted-foreground whitespace-nowrap">
                       De
                     </Label>
                     <Input
                       type="date"
-                      className="h-9 w-32 text-xs"
+                      className="h-9 w-24 sm:w-28 md:w-32 text-xs"
                       value={dateRange.start}
                       onChange={(e) => {
                         setDateRange((p) => ({ ...p, start: e.target.value }));
@@ -402,7 +402,7 @@ export default function FinanceiroPage() {
                     </Label>
                     <Input
                       type="date"
-                      className="h-9 w-32 text-xs"
+                      className="h-9 w-24 sm:w-28 md:w-32 text-xs"
                       value={dateRange.end}
                       onChange={(e) => {
                         setDateRange((p) => ({ ...p, end: e.target.value }));
@@ -418,7 +418,7 @@ export default function FinanceiroPage() {
                         setDateRange({ start: "", end: "" });
                         setPage(1);
                       }}
-                      className="h-9 px-2 text-[10px] font-bold uppercase tracking-widest text-destructive hover:bg-destructive/5"
+                      className="h-9 px-2 text-[10px] font-bold uppercase tracking-widest text-destructive hover:bg-destructive/5 min-h-9"
                     >
                       Limpar
                     </Button>