-
Notifications
You must be signed in to change notification settings - Fork 7.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
perf: Remove tRPC event call from server-side props #17205
Conversation
): Promise<string | undefined> { | ||
const eventTypeMetadataQuery = await prisma.eventType.findUnique({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed this extra roundtrip to the DB to get the metadata since we already had it.
Graphite Automations"Add foundation team as reviewer" took an action on this PR • (10/20/24)1 reviewer was added to this PR based on Keith Williams's automation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know it works 😎
E2E results are ready! |
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 2 Skipped Deployments
|
What does this PR do?
This is broken off from #17194 (credit to @emrysal) since that PR is in draft and has a bunch of typing errors + it's more widespread changes. Keeping this isolated so we can test the difference in perf.
Mandatory Tasks (DO NOT REMOVE)
How should this be tested?