Move vehicle utilization filtering to database query #15

New Issue

kyle · 2026-02-08T06:44:20-08:00

kyle commented

2026-02-08 06:44:20 -08:00

Context: vehicles.service.ts getUtilization() fetches all events then filters for past events in application code. This should be done at the database level.

File: backend/src/vehicles/vehicles.service.ts (lines 116-143)

Suggested fix:

Add where: { endTime: { lt: new Date() } } to the Prisma query
Let PostgreSQL handle the filtering instead of fetching all events to Node.js

Effort: Small

**Context:** `vehicles.service.ts` `getUtilization()` fetches all events then filters for past events in application code. This should be done at the database level. **File:** `backend/src/vehicles/vehicles.service.ts` (lines 116-143) **Suggested fix:** - Add `where: { endTime: { lt: new Date() } }` to the Prisma query - Let PostgreSQL handle the filtering instead of fetching all events to Node.js **Effort:** Small

kyle referenced this issue

2026-02-08 06:44:30 -08:00

Code Efficiency #8

kyle referenced this issue from a commit

2026-02-08 07:07:27 -08:00

refactor: code efficiency improvements (Issues #9-13, #15, #17-20)

kyle commented

2026-02-08 07:09:58 -08:00

Moved vehicle utilization event filtering to the Prisma query with .

kyle closed this issue

2026-02-08 07:09:58 -08:00

Sign in to join this conversation.