cal.com
cal.com copied to clipboard
perf: server-side fetching for all pages in `settings/admin`
What does this PR do?
- Fixes CAL-4311
Mandatory Tasks (DO NOT REMOVE)
- [x] I have self-reviewed the code (A decent size PR without self-review might be rejected).
- [x] N/A - I have added a Docs issue here if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
- [x] I confirm automated tests are in place that prove my fix is effective or that my feature works.
How should this be tested?
Navigate to:
Page Router:
- settings/admin/apps/[category]
- settings/admin/apps
- settings/admin/flags
- settings/admin/lockedSMS
App Router:
- /future/settings/admin/apps/[category]
- /future/settings/admin/apps
- /future/settings/admin/flags
- /future/settings/admin/lockedSMS
Graphite Automations
"Add consumer team as reviewer" took an action on this PR • (09/11/24)
1 reviewer was added to this PR based on Keith Williams's automation.
"Add foundation team as reviewer" took an action on this PR • (09/11/24)
1 reviewer was added to this PR based on Keith Williams's automation.
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 | Sep 17, 2024 3:36am | |
| calcom-web-canary | ⬜️ Ignored (Inspect) | Visit Preview | Sep 17, 2024 3:36am |
@sean-brydon Hey Sean, thanks for the review! I added revalidation logic
This PR is being marked as stale due to inactivity.
This PR is being marked as stale due to inactivity.