Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Getting Started Docs: Improve "Project Structure" page #72399

Merged
merged 12 commits into from
Nov 7, 2024
Merged

Getting Started Docs: Improve "Project Structure" page #72399

merged 12 commits into from
Nov 7, 2024

Conversation

delbaoliveira
Copy link
Contributor

@delbaoliveira delbaoliveira commented Nov 6, 2024 •
edited
Loading

Closes:

This PR:

  • Splits the page between the app/pages and the router docs.
  • Merge the colocation page from BYA.

@ijjk ijjk added created-by: Next.js DevEx team PRs by the DX team. Documentation Related to Next.js' official documentation. labels Nov 6, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 6, 2024 •
edited
Loading

All broken links are now fixed, thank you!

@delbaoliveira delbaoliveira changed the title Docs: Split project structure page between app and pages docs Getting Started Docs: Improve "Project Structure" page Nov 6, 2024
@delbaoliveira delbaoliveira marked this pull request as ready for review November 7, 2024 09:25
@delbaoliveira delbaoliveira merged commit 224447c into canary Nov 7, 2024
52 checks passed
@delbaoliveira delbaoliveira deleted the docs-project-structure branch November 7, 2024 09:34
@mischnic mischnic mentioned this pull request Nov 7, 2024
Merged
delbaoliveira pushed a commit that referenced this pull request Nov 7, 2024
@mischnic
Add missing closing </AppOnly> (#72453)
be40adb 
The MDX was invalid after #72399
causing next-site deploy failures

(I'm not entirely sure if this is where the `</AppOnly>` should go, feel
free to correct that)

Closes PACK-3416
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@molebox molebox molebox approved these changes

Assignees
No one assigned
Labels
created-by: Next.js DevEx team PRs by the DX team. Documentation Related to Next.js' official documentation. locked
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@delbaoliveira @molebox @ijjk