Skip to content

Conversation

@vicb
Copy link
Contributor

@vicb vicb commented May 14, 2025

Summary

Screenshots (optional)

image

Documentation checklist

@hyperlint-ai
Copy link
Contributor

hyperlint-ai bot commented May 14, 2025

Howdy and thanks for contributing to our repo. The Cloudflare team reviews new, external PRs within two (2) weeks. If it's been two weeks or longer without any movement, please tag the PR Assignees in a comment.

We review internal PRs within 1 week. If it's something urgent or has been sitting without a comment, start a thread in the Developer Docs space internally.


PR Change Summary

Enhanced Next.js documentation to include support details for various features in the Cloudflare OpenNext adapter.

  • Added a section detailing Next.js features supported by the Cloudflare adapter
  • Included a table listing supported features and their status
  • Clarified the deployment process for Next.js projects on Cloudflare Workers

Modified Files

  • src/content/docs/workers/frameworks/framework-guides/nextjs.mdx

How can I customize these reviews?

Check out the Hyperlint AI Reviewer docs for more information on how to customize the review.

If you just want to ignore it on this PR, you can add the hyperlint-ignore label to the PR. Future changes won't trigger a Hyperlint review.

Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add hyperlint-ignore to the PR to ignore the link check for this PR.

@vicb vicb requested a review from irvinebroque May 14, 2025 12:55
@github-actions
Copy link
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/docs/workers/frameworks/ @igorminar, @cloudflare/wrangler, @aninibread, @GregBrimble, @ToriLindsay, @cloudflare/pcx-technical-writing

@github-actions
Copy link
Contributor

github-actions bot commented May 14, 2025

Copy link
Collaborator

@kodster28 kodster28 left a comment

Choose a reason for hiding this comment

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

Maybe also recommend adding links to each feature in NextJS?

| Image optimization | 🟢 supported | Supported via [Cloudflare Images](https://developers.cloudflare.com/images/) |
| Partial Prerendering (PPR) | 🟢 supported | PPR is experimental in Next.js |
| Composable Caching ('use cache') | 🟢 supported | Composable Caching is experimental in Next.js |
| Node.js in Middleware | ⚪ not yet supported | Node.js middleware introduced in 15.2 are not yet supported |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should maybe format to be more like https://developers.cloudflare.com/data-localization/compatibility/ throughout?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I based the format on https://developers.cloudflare.com/workers/runtime-apis/nodejs/#supported-nodejs-apis but I agree that a documented/consistent format across the site would be better idea

@vicb
Copy link
Contributor Author

vicb commented May 15, 2025

Maybe also recommend adding links to each feature in NextJS?

I thought about that (we do have links on opennext.js.org/cloudflare)

But decided otherwise here:

  • markdown tables are hard too edit when the columns are too wide (making rows span multiple lines)
  • I don't expect users to come here to learn about Next.js features
  • IMO the main purpose of this table is to show that we are mostly green

But I dont't have a strong opinion, can be convinced to add them!

@kodster28
Copy link
Collaborator

Maybe also recommend adding links to each feature in NextJS?

I thought about that (we do have links on opennext.js.org/cloudflare)

But decided otherwise here:

  • markdown tables are hard too edit when the columns are too wide (making rows span multiple lines)
  • I don't expect users to come here to learn about Next.js features
  • IMO the main purpose of this table is to show that we are mostly green

But I dont't have a strong opinion, can be convinced to add them!

Makes sense to me! You know the audience here better than I do.

@vicb
Copy link
Contributor Author

vicb commented May 16, 2025

@kodster28 Thanks for the feedback. I am going to merge as is for now.

@vicb vicb merged commit 5d0ba8d into production May 16, 2025
11 checks passed
@vicb vicb deleted the vicb/feat-nextjs branch May 16, 2025 19:09
sdnts pushed a commit to sdnts/cloudflare-docs that referenced this pull request Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product:workers Related to Workers product size/s

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants