Pages x Node.js => OpenNext #17123
Merged
Pages x Node.js => OpenNext #17123
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
irvinebroque
requested review from
IgorMinar,
dario-piotrowicz,
jculvey,
aninibread,
GregBrimble,
kodster28 and
a team
as code owners
github-actions
bot
assigned aninibread, dario-piotrowicz, GregBrimble, IgorMinar, jculvey and kodster28
16 tasks
Deploying cloudflare-docs with
|
| Latest commit: |
3b07cc2
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://972c42b8.cloudflare-docs-7ou.pages.dev |
| Branch Preview URL: | https://bib-opennext.cloudflare-docs-7ou.pages.dev |
|
Files with changes (up to 15) |
src/content/docs/pages/framework-guides/nextjs/ssr/get-started.mdx
Outdated
Show resolved
Hide resolved
tanushree-sharma
approved these changes
Sep 26, 2024
….mdx Co-authored-by: Tanushree <[email protected]>
![hyperlint-ai[bot]](https://avatars.githubusercontent.com/in/718456?s=60&v=4){kind=small}
| @@ -13,6 +13,12 @@ import { PackageManagers } from "~/components"; | |||
|
|
|||
| Learn how to deploy full-stack (SSR) Next.js apps to Cloudflare Pages. | |||
|
|
|||
| :::note | |||
| You can now also deploy Next.js apps to [Cloudflare Workers](https://developers.cloudflare.com/workers/frameworks/), including apps that use the Node.js "runtime" from Next.js. This allows you to use the [Node.js APIs that Cloudflare Workers provides](/workers/runtime-apis/nodejs/#built-in-nodejs-runtime-apis), and ensures compatibility with a broader set of Next.js features and rendering modes. | |||
There was a problem hiding this comment.
Suggested change
| You can now also deploy Next.js apps to [Cloudflare Workers](https://developers.cloudflare.com/workers/frameworks/), including apps that use the Node.js "runtime" from Next.js. This allows you to use the [Node.js APIs that Cloudflare Workers provides](/workers/runtime-apis/nodejs/#built-in-nodejs-runtime-apis), and ensures compatibility with a broader set of Next.js features and rendering modes. | |
| You can now also deploy Next.js apps to [Cloudflare Workers](https://developers.cloudflare.com/workers/frameworks/), including apps that use the Node.js "runtime" from Next.js. This allows you to use the [Node.js APIs that Cloudflare Workers provides](/workers/runtime-apis/nodejs/#built-in-nodejs-runtime-apis), and ensures compatibility with a broader set of Next.js features and rendering modes. |
Issues:
- Style Guide - (Terms-error) Use 'Cloudflare' instead of 'cloudflare'.
Fix Explanation:
The only issue identified is the capitalization of 'Cloudflare'. The correction involves capitalizing 'Cloudflare' to adhere to the style guide. The rest of the sentence is clear and does not require any additional changes. If the style guide is updated or if there are other terms that need to be checked, further review might be necessary.
ToriLindsay
approved these changes
Sep 26, 2024
| @@ -13,6 +13,12 @@ import { PackageManagers } from "~/components"; | |||
|
|
|||
| Learn how to deploy full-stack (SSR) Next.js apps to Cloudflare Pages. | |||
|
|
|||
| :::note | |||
| You can now also deploy Next.js apps to [Cloudflare Workers](https://developers.cloudflare.com/workers/frameworks/), including apps that use the Node.js "runtime" from Next.js. This allows you to use the [Node.js APIs that Cloudflare Workers provides](/workers/runtime-apis/nodejs/#built-in-nodejs-runtime-apis), and ensures compatibility with a broader set of Next.js features and rendering modes. | |||
There was a problem hiding this comment.
Suggested change
| You can now also deploy Next.js apps to [Cloudflare Workers](https://developers.cloudflare.com/workers/frameworks/), including apps that use the Node.js "runtime" from Next.js. This allows you to use the [Node.js APIs that Cloudflare Workers provides](/workers/runtime-apis/nodejs/#built-in-nodejs-runtime-apis), and ensures compatibility with a broader set of Next.js features and rendering modes. | |
| You can also deploy Next.js apps to [Cloudflare Workers](https://developers.cloudflare.com/workers/frameworks/), including apps that use the Node.js "runtime" from Next.js. This allows you to use the [Node.js APIs that Cloudflare Workers provides](/workers/runtime-apis/nodejs/#built-in-nodejs-runtime-apis), and ensures compatibility with a broader set of Next.js features and rendering modes. |
There was a problem hiding this comment.
I've just found that terms like "now" or announcement language tends to age poorly if they're not updated often and I confirmed with another TW that we steer away from that.
| :::note | ||
| You can now also deploy Next.js apps to [Cloudflare Workers](https://developers.cloudflare.com/workers/frameworks/), including apps that use the Node.js "runtime" from Next.js. This allows you to use the [Node.js APIs that Cloudflare Workers provides](/workers/runtime-apis/nodejs/#built-in-nodejs-runtime-apis), and ensures compatibility with a broader set of Next.js features and rendering modes. | ||
|
|
||
| Refer to the [OpenNext docs for the `@opennextjs/cloudflare` adapter](https://opennext.js.org/cloudflare) to learn how to get started. |
There was a problem hiding this comment.
Suggested change
| Refer to the [OpenNext docs for the `@opennextjs/cloudflare` adapter](https://opennext.js.org/cloudflare) to learn how to get started. | |
| Learn how to get started in the [OpenNext documentation for the `@opennextjs/cloudflare` adapter] (https://opennext.js.org/cloudflare) |
patriciasantaana
pushed a commit
that referenced
this pull request
Sep 26, 2024
* Pages x Node.js => OpenNext refs opennextjs/docs#3 * Update src/content/docs/pages/framework-guides/nextjs/ssr/get-started.mdx Co-authored-by: Tanushree <[email protected]> --------- Co-authored-by: Tanushree <[email protected]>
elithrar
pushed a commit
that referenced
this pull request
Oct 15, 2024
* Pages x Node.js => OpenNext refs opennextjs/docs#3 * Update src/content/docs/pages/framework-guides/nextjs/ssr/get-started.mdx Co-authored-by: Tanushree <[email protected]> --------- Co-authored-by: Tanushree <[email protected]>
maheshwarip
pushed a commit
that referenced
this pull request
Dec 2, 2024
* Pages x Node.js => OpenNext refs opennextjs/docs#3 * Update src/content/docs/pages/framework-guides/nextjs/ssr/get-started.mdx Co-authored-by: Tanushree <[email protected]> --------- Co-authored-by: Tanushree <[email protected]>
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.
refs opennextjs/docs#3