Closed cstrnt closed 2 months ago
The latest updates on your projects. Learn more about Vercel for Git โ๏ธ
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
abby-docs | โ Failed (Inspect) | Aug 28, 2024 9:48pm |
[!CAUTION]
Review failed
The pull request is closed.
The changes involve updates to the Prisma ORM configuration, including modifications to the package.json
scripts and dependencies, enhancements to the Prisma schema with new preview features, and the introduction of several SQL query files for event data retrieval. Additionally, the TIME_INTERVAL
enum in the events library has been updated, and the EventService
class has been refactored to improve event fetching logic.
Files | Change Summary |
---|---|
apps/web/package.json |
Updated db:generate script to include prisma generate --sql . Updated @prisma/client and prisma versions from 5.18.0 to 5.19.0 . |
apps/web/prisma/schema.prisma |
Added "typedSql" to previewFeatures in the generator client block. |
apps/web/prisma/sql/getEventsByTestIdForAllTime.sql , apps/web/prisma/sql/getEventsByTestIdForDay.sql , apps/web/prisma/sql/getEventsByTestIdForLast30Days.sql |
New SQL queries introduced for retrieving event data based on test ID, event type, and time intervals. |
apps/web/src/lib/events.ts |
Added LAST_30_DAYS to TIME_INTERVAL enum. Modified getFormattingByInterval function. Removed getMSFromSpecialTimeInterval and getLabelsByInterval functions. |
apps/web/src/server/services/EventService.ts |
Updated getEventsByTestId method to include eventType parameter and restructured logic to use raw SQL queries. |
apps/web/src/server/trpc/router/events.ts |
Restructured logic for retrieving and processing event data in the eventRouter . |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
This stack of pull requests is managed by Graphite. Learn more about stacking.
Join @cstrnt and the rest of your teammates on Graphite
Summary by CodeRabbit
New Features
Bug Fixes
Refactor