Worker Previews docs: restructure resources, update cross-references - #31775
yomna-shousha wants to merge 31 commits into
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
|
This PR changes current filenames or deletes current files. Make sure you have redirects set up to cover the following paths:
|
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
|
I have completed my review of PR #31775. My findings are summarized above in the review. Here's a quick recap of what needs maintainer attention:
Everything else looks solid: redirects are in place, MDX syntax is clean, component imports and image paths are correct, and the cross-reference updates are comprehensive. |
This comment was marked as outdated.
This comment was marked as outdated.
- Fix Access link: /cloudflare-one/policies/access/ → /cloudflare-one/access-controls/policies/ - Fix placement link: /workers/configuration/smart-placement/ → /workers/configuration/placement/ - Add missing redirect for deleted /workers/previews/troubleshooting/
…fig push - preview secret put/delete/list/bulk: --name now optional (defaults to git branch) - Added --message and --tag flags to put, delete, and bulk - Updated descriptions to reflect per-deployment patching behavior - preview secret bulk: documented FILE positional arg and null value deletion - preview secret list: noted it reads from latest Preview deployment - Fixed bindings.mdx typo: 'preview base secret put' → 'preview base-config secret put' - Updated recipes.mdx migration table with --name flag - Removed preview base-config push command (not shipping)
…uth for Preview settings
- Rewrite get-started page: cut redundancy, add Steps/DashButton, 3 deploy tabs - Tighten overview: remove isolation matrix from top, simplify limits/URLs - Simplify configuration: shorten callout, collapse repeated wrangler-applies section - Clean up resources: merge bullet list into matrix, remove duplicated sections - Restructure custom-domains: tabs for Dashboard/Wrangler/API, workers.dev toggle steps, Enable for options with screenshots - Add dashboard screenshots (pending rendering verification in prod) - CI/CD page unchanged (already clean)
…, simplify deploy steps
…ig, remove dashboard overlap, add wrangler envs example
4583673 to
a080e74
Compare
Co-authored-by: mvvmm <mvvmm@users.noreply.github.com>
|
Agreed — these five components are page-specific (each used on exactly one page), so they belong as direct imports, not in the The repo's documented convention ( Changes made:
Validation: |
What this PR does
Restructures and expands the Worker Previews documentation.
Changes
New pages:
previews/recipes.mdx— static how-tos: PR preview per branch, persistent staging, data isolation, monorepo, migration from Wrangler environments, API-managed previewspreviews/resources/index.mdx— overview of resource sharing and isolationpreviews/resources/bindings.mdx— per-binding reference with config snippetspreviews/resources/durable-objects.mdx— DO and Containers isolation detailsNew components:
PreviewsConfigDiagram.astro— base configuration diagramPreviewsResourcesDiagram.astro— resource sharing diagramRestructured:
resources.mdxintoresources/directory (index, bindings, durable-objects)previews/commands.mdx→wrangler/commands/workers.mdxUpdated cross-references:
Deleted:
commands.mdx(content moved to wrangler commands)resources.mdx(split into resources/ directory)troubleshooting.mdx(content folded into other pages)Normalized: "Worker Previews" → "Previews" across referenced pages
Not included (separate commits)
Known issues to fix later
/workers/configuration/cloudflare-access/link in index.mdx needs correct path