-
-
Notifications
You must be signed in to change notification settings - Fork 724
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
feat: add clerk integration in /next branch #364
feat: add clerk integration in /next branch #364
Conversation
Someone is attempting to deploy a commit to the Noodle Team on Vercel. A member of the Team first needs to authorize it. |
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.
Thanks for the PR, there are some changes that need to be done however:
- add sign in / sign up pages for clerk.
- add clerk environment variables to env.ts.
- make the middleware only run on routes that start with /app because we wouldn't want to rate limit or check for authentication outside of that.
I also so see that you have also copied over the upstash rate limiting flow, I would change up the PR scope/title to match this then, something like feat: adds clerk & upstash integrations
@ixahmedxi I have made the changes you suggested, please let me know if anything more needs to be done:) |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Resolves #363
This PR will add the clerk authentication in the /next branch.