-
Notifications
You must be signed in to change notification settings - Fork 402
feat(clerk-js,clerk-react,types): Navigate to tasks on after sign-in/sign-up #5280
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
Conversation
🦋 Changeset detectedLatest commit: b10bf3f The changes in this PR will be included in the next version bump. This PR includes changesets to release 22 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
SignIn/SignUp componentSignIn/SignUp component
c15f966 to
fafd3b7
Compare
63ef78b to
1770981
Compare
1770981 to
83ae82c
Compare
83ae82c to
e09b5e7
Compare
7a6ad6d to
82034aa
Compare
SignIn/SignUp component82034aa to
5e62dd7
Compare
e10d5a7 to
29d71c8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to 🚢 Some minor comments are
- Update changeset to include
@clerk/clerk-react - Avoid logging for
<SignIn/>being unable to render when user can access'/sign-in/add-organization'
|
I'm currently doing more tests on SSO callbacks + combined flows that trigger |
- Use the typed constant when declaring Route components to catch breaking changes on `path` changes - Call `setComponentNavigationContext` within SignIn/SignUp root components to trigger cleanup on unmount, instead of relying on `BaseRouter` - Remove cleanup for `Clerk.#componentNavigationContext` withi `Clerk.#handlePendingSession`
3722d1f to
c95edab
Compare
|
Regarding the SSO callback flow -> It navigates to task when FAPI redirects to However, it's not working if the sign-in is already created on FAPI, and it's redirecting to the There are two ways to approach (in a next PR):
|
Description
Introduces navigation to tasks after sign-in/sign-up for UI components and custom flows:
CleanShot.2025-03-06.at.19.33.15-converted.mp4
Developer flow
It doesn't require changes on the
setActivesignature, since the navigation is triggered withinClerkonce the session transitions topendingWhat's next:
Clerk.loadoption (tasksUrl) for resolving URLs for tasks on custom flowsChecklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change