Skip to content

Conversation

@ben-fornefeld
Copy link
Member

@ben-fornefeld ben-fornefeld commented Nov 11, 2025

Note

Strips verbose debug logs and refactors team metadata retrieval from cookies, updating imports and a sign-in redirect usage.

  • Server utils (src/lib/utils/server.ts):
    • Replace getTeamMetadataFromCookiesMemo/cache flow with simplified getTeamMetadataFromCookies that directly validates cookie ID/slug.
  • Team resolution (src/server/team/resolve-user-team.ts):
    • Use new getTeamMetadataFromCookies and remove debug logs; logic unchanged otherwise.
  • Routes:
    • src/app/dashboard/account/route.ts: remove debug logs; use AUTH_URLS.SIGN_IN constant for unauth redirect; preserve query params; set team cookies and redirect to resolved account settings.
    • src/app/dashboard/route.ts: remove debug logs; unchanged redirect logic, still resolves team and redirects based on tab.

Written by Cursor Bugbot for commit 6ac2f75. This will update automatically on new commits. Configure here.

…eamline team metadata retrieval

- Eliminated debug logging statements from `dashboard/route.ts` and `account/route.ts` to reduce noise and improve performance.
- Updated `getTeamMetadataFromCookiesMemo` to `getTeamMetadataFromCookies` for consistency and clarity in team metadata retrieval logic in `server/team/resolve-user-team.ts`.
- Simplified the logic for resolving team metadata from cookies, enhancing maintainability.
@vercel
Copy link

vercel bot commented Nov 11, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
web Ready Ready Preview Comment Nov 11, 2025 3:49pm
web-juliett Ready Ready Preview Comment Nov 11, 2025 3:49pm

@ben-fornefeld ben-fornefeld merged commit 4d66499 into main Nov 11, 2025
6 checks passed
@ben-fornefeld ben-fornefeld deleted the chore-remove-debug-logs branch November 11, 2025 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants