[Pages] Simplify Next.js docs (and port over content from next-on-pages repo)
#15995
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
This PR changes current filenames or deletes current files. Make sure you have redirects set up to cover the following paths:
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Deploying cloudflare-docs with
|
| Latest commit: |
2f3459d
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://8ae31fa1.cloudflare-docs-7ou.pages.dev |
| Branch Preview URL: | https://simplify-next-js.cloudflare-docs-7ou.pages.dev |
Closed
irvinebroque
reviewed
Aug 6, 2024
irvinebroque
requested review from
jculvey,
a team and
GregBrimble
as code owners
content/pages/framework-guides/nextjs/ssr/supported-features.md
Outdated
Show resolved
Hide resolved
content/pages/framework-guides/nextjs/ssr/supported-features.md
Outdated
Show resolved
Hide resolved
content/pages/framework-guides/nextjs/ssr/supported-features.md
Outdated
Show resolved
Hide resolved
irvinebroque
force-pushed
the
simplify-next-js
branch
from
f98a131 to
abff8a2
Compare
irvinebroque
reviewed
Aug 9, 2024
irvinebroque
reviewed
Aug 9, 2024
james-elicx
reviewed
Aug 9, 2024
There was a problem hiding this comment.
can you add back the advanced usage section you're removing in next-on-pages too?
irvinebroque
reviewed
Aug 9, 2024
kodster28
commented
Aug 9, 2024
content/pages/framework-guides/nextjs/ssr/supported-features.md
Outdated
Show resolved
Hide resolved
content/pages/framework-guides/nextjs/ssr/supported-features.md
Outdated
Show resolved
Hide resolved
content/pages/framework-guides/nextjs/ssr/supported-features.md
Outdated
Show resolved
Hide resolved
content/pages/framework-guides/nextjs/ssr/supported-features.md
Outdated
Show resolved
Hide resolved
content/pages/framework-guides/nextjs/ssr/supported-features.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Kody Jackson <[email protected]>
Co-authored-by: Kody Jackson <[email protected]>
Co-authored-by: Kody Jackson <[email protected]>
Co-authored-by: Kody Jackson <[email protected]>
dcpena
approved these changes
Aug 9, 2024
irvinebroque
approved these changes
Aug 9, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
An attempt to address the comment at cloudflare/next-on-pages#850 (review), where we provide a streamlined guide (as suggested in #15971), but then do the following:
Fullstack (SSR)get startedguide onto their own pagesnext-on-pagesrepo, withcachingas the first example page.Might be slightly misunderstanding how those
next-on-pagesdocs are scoped... my assumption here is that they're mostly relevant to SSR vs static, but let me know if that's an incorrect assumption.Branched off #15971 since it was a good start on the consolidation.