Skip to content

perf: Remove QueryCell usage in availability view#20176

Merged
hbjORbj merged 2 commits into
mainfrom
perf/availability-view-query-cell
Mar 18, 2025
Merged

perf: Remove QueryCell usage in availability view#20176
hbjORbj merged 2 commits into
mainfrom
perf/availability-view-query-cell

Conversation

@hbjORbj
Copy link
Copy Markdown
Contributor

@hbjORbj hbjORbj commented Mar 18, 2025

What does this PR do?

  • We figured that there's a small delay between RSC component fetch and TRPC calls when QueryCell is used (likely due to @calcom/ui imports)

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A - I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  • Test /availability (I already did)

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 18, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Mar 18, 2025 7:19pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Mar 18, 2025 7:19pm

@graphite-app graphite-app Bot requested a review from a team March 18, 2025 19:06
@keithwillcode keithwillcode added consumer core area: core, team members only labels Mar 18, 2025
@dosubot dosubot Bot added the performance area: performance, page load, slow, slow endpoints, loading screen, unresponsive label Mar 18, 2025
@hbjORbj hbjORbj changed the title perf: availability view query cell perf: Remove QueryCell usage in availability view Mar 18, 2025
@hbjORbj hbjORbj enabled auto-merge (squash) March 18, 2025 19:07
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Mar 18, 2025

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (03/18/25)

1 reviewer was added to this PR based on Keith Williams's automation.

@github-actions
Copy link
Copy Markdown
Contributor

E2E results are ready!

@hbjORbj hbjORbj merged commit cc1b6ba into main Mar 18, 2025
@hbjORbj hbjORbj deleted the perf/availability-view-query-cell branch March 18, 2025 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

consumer core area: core, team members only performance area: performance, page load, slow, slow endpoints, loading screen, unresponsive ready-for-e2e

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants