-
Notifications
You must be signed in to change notification settings - Fork 27k
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
Update example with-mongodb to add both Pages & App examples #64130
Conversation
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.
Thank you for updating this example Jesse and Justin! 🙏🏼
Could you please resolve the merge conflict?
Thank you for checking this out @delbaoliveira. I've addressed the conflicts and proceeded to merge. Please let me know if there is anything else needed. |
It looks like the lint is mad about your HEAD being detached → https://github.com/vercel/next.js/actions/runs/10085328885/job/27885936674?pr=64130.
|
@samcx All changes are committed, please try again. Thanks! |
Giving the same issue... hmmm |
@codeSTACKr I'm not sure if our CI is not happy because you're trying to merge your fork's canary to our canary. Can you try using a different branch? |
@samcx Ok, I'll give that a try. |
Updated all dependencies and added an app route to demonstrate both pages an app directories.