fix start date change not possible in time filter (#2749)

* fix start date change not possible in time filter * use satisfies instead of as
Helicone · Oct 9, 2024 · 5e48cd0 · 5e48cd0
1 parent cfe1b93
commit 5e48cd0
Showing 1 changed file with 20 additions and 16 deletions.
diff --git a/web/components/shared/themed/themedTimeFilterShadCN.tsx b/web/components/shared/themed/themedTimeFilterShadCN.tsx
@@ -234,25 +234,29 @@ export function ThemedTimeFilterShadCN({
               defaultMonth={date?.from}
               selected={date}
               onSelect={(newDate) => {
-                if (newDate?.from && newDate?.to && date?.from && date?.to) {
+                if (newDate?.from || newDate?.to) {
                   const newDateRange = {
-                    from: new Date(
-                      newDate.from.getFullYear(),
-                      newDate.from.getMonth(),
-                      newDate.from.getDate(),
-                      date.from.getHours(),
-                      date.from.getMinutes()
-                    ),
-                    to: new Date(
-                      newDate.to.getFullYear(),
-                      newDate.to.getMonth(),
-                      newDate.to.getDate(),
-                      date.to.getHours(),
-                      date.to.getMinutes()
-                    ),
+                    from: newDate?.from
+                      ? new Date(
+                          newDate.from.getFullYear(),
+                          newDate.from.getMonth(),
+                          newDate.from.getDate(),
+                          date?.from?.getHours() ?? 0,
+                          date?.from?.getMinutes() ?? 0
+                        )
+                      : date?.from,
+                    to: newDate?.to
+                      ? new Date(
+                          newDate.to.getFullYear(),
+                          newDate.to.getMonth(),
+                          newDate.to.getDate(),
+                          date?.to?.getHours() ?? 23,
+                          date?.to?.getMinutes() ?? 59
+                        )
+                      : date?.to,
                   };
 
-                  handleDateChange(newDateRange);
+                  handleDateChange(newDateRange satisfies DateRange);
                 }
               }}
               numberOfMonths={2}