Skip to content

chore: next.js dir clarification #2269

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

Merged
merged 2 commits into from
Aug 13, 2025
Merged

chore: next.js dir clarification #2269

merged 2 commits into from
Aug 13, 2025

Conversation

Milk-Tea
Copy link
Contributor

@Milk-Tea Milk-Tea commented Aug 8, 2025

As a user following this quickstart, it was unclear that the guide uses an alias.
The config and middleware files were to be placed in the root, but if a user has chosen to use a src/ directory, then the user will run into problems.

This change is to prevent such problems from arising.

@Milk-Tea Milk-Tea requested review from vinckr and aeneasr as code owners August 8, 2025 16:53
@CLAassistant
Copy link

CLAassistant commented Aug 8, 2025

CLA assistant check
All committers have signed the CLA.

@vinckr vinckr changed the title Update 16_nextjs_app_router.mdx chore: next.js dir clarification Aug 13, 2025
Copy link
Member

@vinckr vinckr left a comment

Choose a reason for hiding this comment

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

thanks so much for your contribution 🚀

I fixed the style a bit

@vinckr vinckr enabled auto-merge (squash) August 13, 2025 13:07
@vinckr vinckr merged commit ef7a2e0 into ory:master Aug 13, 2025
8 checks passed
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.

3 participants