-
Notifications
You must be signed in to change notification settings - Fork 432
fix(shared): Use static value for authenticated in useOrganizationCreationDefaultsCacheKeys
#7724
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
base: main
Are you sure you want to change the base?
fix(shared): Use static value for authenticated in useOrganizationCreationDefaultsCacheKeys
#7724
Conversation
…CreationDefaultsCacheKeys` Adds comments that better describe the intended use of `authenticated` in `createCacheKeys`. Then intention is to have all "authenticated queries" (queries that depend on user or org) to be cleared from the cache in order to avoid rendering stale data about the user after they have signed out.
|
|
@panteliselef is attempting to deploy a commit to the Clerk Production Team on Vercel. A member of the Team first needs to authorize it. |
📝 WalkthroughWalkthroughThese changes modify the organization creation defaults functionality by adding documentation to the cache keys parameter, updating authentication handling to be always enabled instead of conditional on user presence, and introducing sign-out detection with automatic query clearing. The modifications span documentation updates, cache key value assignments, and integration of sign-out lifecycle management into the hook's execution flow. 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
|
@anagstef Do we trust this contributor? Shall I let the workflows to run? |
@clerk/agent-toolkit
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/dev-cli
@clerk/expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
Description
Improves
useOrganizationCreationDefaultsCacheKeysand adds comments that better describe the intended use ofauthenticatedincreateCacheKeys.Then intention is to have all "authenticated queries" (queries that depend on user or org) to be cleared from the cache in order to avoid rendering stale data about the user after they have signed out.
Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.