cal.com icon indicating copy to clipboard operation
cal.com copied to clipboard

perf: cache all server calls used in /members pages

Open hbjORbj opened this issue 11 months ago • 3 comments

What does this PR do?

  • In /settings/platform/members, Use OrganizationRepository methods directly (prisma calls) and cache these calls
  • In /settings/organization/members, Use OrganizationRepository methods directly (prisma calls) and cache these calls

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 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.
  • [x] I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  • Test /settings/platform/members
  • Test /settings/organization/members

hbjORbj avatar May 30 '25 01:05 hbjORbj

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 Jun 26, 2025 7:05am
cal-eu ⬜️ Ignored (Inspect) Visit Preview Jun 26, 2025 7:05am

vercel[bot] avatar May 30 '25 01:05 vercel[bot]

Graphite Automations

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

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

"Add foundation team as reviewer" took an action on this PR • (06/12/25)

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

graphite-app[bot] avatar May 30 '25 01:05 graphite-app[bot]

E2E results are ready!

github-actions[bot] avatar Jun 12 '25 16:06 github-actions[bot]

No security or compliance issues detected. Reviewed everything up to 429721508b68da771e6cf0ee45f823188b7d5fcf.

Security Overview
  • 🔎 Scanned files: 47 changed file(s)
Detected Code Changes
Change Type Relevant files
Enhancement ► cache/.ts
    Add cache functionality for API keys, attributes, organizations and teams
► teams/components/
.tsx
    Update team components to use new cache methods
► organizations/pages/*.tsx
    Update organization pages to use new cache methods
Refactor ► server/repository/organization.ts
    Add getTeams method
► organizations/getTeams.handler.ts
    Update to use repository pattern
Configuration changes ► constants.ts
    Add NEXTJS_CACHE_TTL constant

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

delve-auditor[bot] avatar Jun 25 '25 06:06 delve-auditor[bot]

This PR is being marked as stale due to inactivity.

github-actions[bot] avatar Jul 11 '25 00:07 github-actions[bot]

closing due to staleness

keithwillcode avatar Aug 12 '25 18:08 keithwillcode