Skip to content

Add Cloudflare adapter section to advanced routing docs#13912

Merged
ArmandPhilippot merged 8 commits into
mainfrom
cloudflare-handlers
May 29, 2026
Merged

Add Cloudflare adapter section to advanced routing docs#13912
ArmandPhilippot merged 8 commits into
mainfrom
cloudflare-handlers

Conversation

@matthewp
Copy link
Copy Markdown
Contributor

Description (required)

Documents the new fetch handlers for Cloudflare.

Related issues & labels (optional)

withastro/astro#16729

@astrobot-houston
Copy link
Copy Markdown
Contributor

astrobot-houston commented May 19, 2026

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
en/reference/experimental-flags/advanced-routing.mdx Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Copy link
Copy Markdown
Member

@florian-lefebvre florian-lefebvre left a comment

Choose a reason for hiding this comment

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

I think this section should mention how that interacts with main in wrangler.jsonc. Like you probably don't want to use this and @astrojs/cloudflare/utils/handler at the same time

@ematipico ematipico added the merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) label May 19, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented May 19, 2026

Deploy Preview for astro-docs-2 ready!

Name Link
🔨 Latest commit 6ef6269
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/6a0c69ea61f33b00083a384d
😎 Deploy Preview https://deploy-preview-13912--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 19, 2026

Deploy Preview for astro-docs-2 ready!

Name Link
🔨 Latest commit 27df4ea
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/6a1999199877720008703217
😎 Deploy Preview https://deploy-preview-13912--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@matthewp
Copy link
Copy Markdown
Contributor Author

@florian-lefebvre I added a short explanation in 8f90f76. Let me know if you think it deserves more of an explanation, hard to describe.

Comment thread src/content/docs/en/reference/experimental-flags/advanced-routing.mdx Outdated
Comment thread src/content/docs/en/reference/experimental-flags/advanced-routing.mdx Outdated
Comment thread src/content/docs/en/reference/experimental-flags/advanced-routing.mdx Outdated
Comment thread src/content/docs/en/reference/experimental-flags/advanced-routing.mdx Outdated
Copy link
Copy Markdown
Member

@florian-lefebvre florian-lefebvre left a comment

Choose a reason for hiding this comment

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

I think that's better thank you!

Comment thread src/content/docs/en/reference/experimental-flags/advanced-routing.mdx Outdated
Copy link
Copy Markdown
Member

@ArmandPhilippot ArmandPhilippot left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM!

Copy link
Copy Markdown
Member

@yanthomasdev yanthomasdev left a comment

Choose a reason for hiding this comment

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

LGTM!

@ArmandPhilippot ArmandPhilippot added this to the 6.4 milestone May 29, 2026
@ArmandPhilippot ArmandPhilippot added the minor-release For the next minor release; in the milestone, "merge queue" = approved for merging on release day. label May 29, 2026
@ArmandPhilippot
Copy link
Copy Markdown
Member

It seems we forgot to put this in the 6.4 milestone 🤦🏽 I'm merging this now!

@ArmandPhilippot ArmandPhilippot merged commit ea8c27b into main May 29, 2026
12 checks passed
@ArmandPhilippot ArmandPhilippot deleted the cloudflare-handlers branch May 29, 2026 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) minor-release For the next minor release; in the milestone, "merge queue" = approved for merging on release day.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants