-
Notifications
You must be signed in to change notification settings - Fork 17
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
Next15 support #125
Next15 support #125
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
3294ff5
to
3f2dee3
Compare
commit: |
ffdf259
to
f1626fd
Compare
Iterating separately from Chadwick's branch, #118 |
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.
So we've gotten some testing with Next 15 and the alpha version with this code -- as long as we've also manually tested that Next 14 still works (after adding all the await
s) I'm down to ship it
Just manually tested, looks good with a Next.js 14 site. |
6ac4bca
to
a42eded
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.
Improve docs @thomasballinger
@@ -28,10 +26,10 @@ const isSignInPage = createRouteMatcher(["/signin"]); | |||
const isProtectedRoute = createRouteMatcher(["/product(.*)"]); | |||
|
|||
export default convexAuthNextjsMiddleware((request, { convexAuth }) => { |
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.
The arrow function is lacking async
here.
It should be
export default convexAuthNextjsMiddleware(async (request, { convexAuth }) => {
No description provided.