Bug in Convex Auth: shouldHandleCode does not exist
Followed the docs to allow handling oauth flow for custom integrations. But it says this does not exist.
Also, is the @convex-dev/react-query still maintained? Or should we be using something else. Primarily use this to get loading states for mutations and actions.
10 Replies
Convex react query is still maintained, just no recent commits. 0.0.84 of convex auth was released specifically to address this, I’ll take a look as soon as I’m back to my machine.
Right here: https://github.com/get-convex/convex-auth/blob/f9db58c8ee45ec528e168f792a1621459fd3224e/src/react/index.tsx#L93
I see the version confirmed in your screenshots, but is there any chance the actual running version is different?
Our repo uses the same version all across. We specially use sheriff in our repo to ensure all apps and packages are using the same versions of any installed package. We run a postinstall job to actually enforce this. So there really is no chance of a different running version. The post install job does prompt you to choose a version to resolve if it detects a drift.
Using a different component than
ConvexAuthProvider specifically?
Here's the types for that published version on unpkg: https://app.unpkg.com/@convex-dev/auth@0.0.84/files/dist/react/index.d.tsUNPKG
The CDN for everything on npm
The error there says only children and client are accepted, which doesn't match ConvexAuthProvider even before this prop was added.
I am using the ConvexAuthNextjsProvider. Error still persists. Is it perhaps not available for this?
Ah, it’s missing from the server provider still. Are you using ConvexAuthNextjsServerProvider in your root layout
Yes.
the component takes children+ client variables, you don't need the "should run code" definition
Yeah it wouldn't go there, it would go in the server provider, but this prop wasn't added there. Fixing now.
@thedevstockgirl fixed in 0.0.87, you can add this prop to the server provider
Thanks so much.