Skip to content

Fix issues with tanstack-start-authkit#249

Open
yongtheskill wants to merge 1 commit intoget-convex:mainfrom
yongtheskill:fix-tanstack-start-authkit
Open

Fix issues with tanstack-start-authkit#249
yongtheskill wants to merge 1 commit intoget-convex:mainfrom
yongtheskill:fix-tanstack-start-authkit

Conversation

@yongtheskill
Copy link

Fixed issues mentioned in #204

  1. For Warning: useRouter must be used inside a <RouterProvider> component: changed Wrap to InnerWrap In router.tsx
  2. Redirect loops when trying to sign out, fixed in @workos/authkit-tanstack-react-start 0.2.0. Updated to latest version.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Copy link
Member

@Nicolapps Nicolapps left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution!

When I try to run the project and log in, I get the following error:

Error: It looks like you forgot to return a response from your server route handler. If you want to defer to the app router, make sure to have a component set in this route.
    at throwRouteHandlerError (/Users/nicolas/Documents/templates/template-tanstack-start-authkit/node_modules/@tanstack/start-server-core/src/createStartHandler.ts:420:11)
    ... 8 lines matching cause stack trace ...
    at async file:///Users/nicolas/Documents/templates/template-tanstack-start-authkit/node_modules/@tanstack/start-plugin-core/dist/esm/dev-server-plugin/plugin.js:64:30 {
  cause: Error: It looks like you forgot to return a response from your server route handler. If you want to defer to the app router, make sure to have a component set in this route.
      at throwRouteHandlerError (/Users/nicolas/Documents/templates/template-tanstack-start-authkit/node_modules/@tanstack/start-server-core/src/createStartHandler.ts:420:11)
      at eval (/Users/nicolas/Documents/templates/template-tanstack-start-authkit/node_modules/@tanstack/start-server-core/src/createStartHandler.ts:445:7)
      at process.processTicksAndRejections (node:internal/process/task_queues:105:5)
      at async next (/Users/nicolas/Documents/templates/template-tanstack-start-authkit/node_modules/@tanstack/start-server-core/src/createStartHandler.ts:461:16)
      at async Object.next (/Users/nicolas/Documents/templates/template-tanstack-start-authkit/node_modules/@tanstack/start-server-core/src/createStartHandler.ts:466:30)
      at async newResponse (/Users/nicolas/Documents/templates/template-tanstack-start-authkit/node_modules/@workos/authkit-tanstack-react-start/dist/server/middleware.js:27:24)
      at async next (/Users/nicolas/Documents/templates/template-tanstack-start-authkit/node_modules/@tanstack/start-server-core/src/createStartHandler.ts:461:16)
      at async startRequestResolver (/Users/nicolas/Documents/templates/template-tanstack-start-authkit/node_modules/@tanstack/start-server-core/src/createStartHandler.ts:248:19)
      at async Object.fetch (/Users/nicolas/Documents/templates/template-tanstack-start-authkit/node_modules/@tanstack/react-start/dist/plugin/default-entry/server.ts:16:14)
      at async file:///Users/nicolas/Documents/templates/template-tanstack-start-authkit/node_modules/@tanstack/start-plugin-core/dist/esm/dev-server-plugin/plugin.js:64:30,
  status: 500,
  statusText: undefined,
  headers: undefined,
  data: undefined,
  body: undefined,
  unhandled: true
}

Is it working on your side?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants