perf(SingleLinePage): don't fetch irrelevant data (#761)

hasadna · May 21, 2024 · 68e35b8 · 68e35b8
1 parent de06832
commit 68e35b8
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/api/useVehicleLocations.ts b/src/api/useVehicleLocations.ts
@@ -166,16 +166,19 @@ export default function useVehicleLocations({
   lineRef,
   operatorRef,
   splitMinutes: split = 1,
+  pause = false,
 }: {
   from: Dateable
   to: Dateable
   lineRef?: number
   operatorRef?: number
   splitMinutes?: false | number
+  pause?: boolean
 }) {
   const [locations, setLocations] = useState<VehicleLocation[]>([])
   const [isLoading, setIsLoading] = useState<boolean[]>([])
   useEffect(() => {
+    if (pause) return
     const range = split ? getMinutesInRange(from, to, split) : [{ from, to }]
     setIsLoading(range.map(() => true))
     const unmounts = range.map(({ from, to }, i) =>

diff --git a/src/hooks/useSingleLineData.ts b/src/hooks/useSingleLineData.ts
@@ -18,7 +18,8 @@ export const useSingleLineData = (lineRef?: number, routeIds?: number[]) => {
     from: +new Date(timestamp).setHours(0, 0, 0, 0),
     to: +new Date(timestamp).setHours(23, 59, 59, 999),
     lineRef,
-    splitMinutes: 20,
+    splitMinutes: 60,
+    pause: !lineRef,
   })
 
   const positions = useMemo(() => {