sanity-migration: point Webflow reference at the Sanity Learn migration guide - #60
Draft
claude[bot] wants to merge 3 commits into
Draft
sanity-migration: point Webflow reference at the Sanity Learn migration guide#60claude[bot] wants to merge 3 commits into
claude[bot] wants to merge 3 commits into
Conversation
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EwCiGS4nB5F1Benh3jYTun
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Webflow → Sanity migration reference to be self-contained and aligned with the repo’s migration-reference format, removing any requirement to fetch external content during execution.
Changes:
- Adds a concise lead-in and expands the Webflow migration guidance with more specific extraction, modeling, and import sequencing notes.
- Standardizes the section structure to match the other platform migration references (audit → extraction → mapping → transform → import → cutover → gotchas → validation).
- Incorporates Webflow-specific operational details (Data API usage, CDN domains, two-pass
_weakreference resolution, cutover checklist).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+29
to
+30
| - **Site structure via API:** `GET /sites/{id}/collections` (field types, reference targets), `/pages` (slugs + SEO metadata — this is your URL inventory, no sitemap XML needed), `/components` (component definitions), `/forms` (field schemas), and `/custom_code` (script inventory; Business plan or higher). Save each response as JSON and inspect before modeling. | ||
| - **Static pages (DOM API):** `GET /v2/pages/{page_id}/dom` returns only editable content nodes — `text`, `image`, and `component-instance` (each with a `propertyOverrides` array of the content set per instance). It carries no layout context; pair it with the HTML export, which is the only source of section structure. |
…PI notation Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EwCiGS4nB5F1Benh3jYTun
This file contains hidden or 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
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.
Requested by Knut Melvær · Slack thread
Before: The Webflow migration reference duplicated a long body of platform guidance, and the first commit on this branch added a "Start Here" block telling the agent to fetch the Sanity Learn course URL — making a network fetch a required step.
After: A self-contained, distilled guide informed by the Sanity Learn Webflow → Sanity course. It is followable with no network — the course URL is kept only as a source/further-reading citation near the top — and it matches the format of the other platform references (
contentful.md,wordpress.md): the same terse, imperative H2 skeleton (What to Determine First→Extraction Paths→Mapping to Sanity→Transformation Notes→Import Order→Static Page and Page Builder Workflow→Cutover→Gotchas→Validation Checklist).Genuinely useful course specifics are folded in (distilled to bullets): the Webflow Data API as the preferred path with ID-to-ID reference joins, the three Webflow CDN domains, the Assets-API blind spot, the two-pass
_weak-placeholder import, deterministictype-idIDs,_key-on-every-array-item,htmlToBlocks()rich-text conversion, DOM-API static-page extraction, page classification (stacked / split / CMS-driven / custom), the eight Webflow content-modeling anti-patterns, forms-as-full-replacement,vercel.jsonredirects, and DNS/404 cutover steps. All agent-specific guidance is retained: reference-ID lookup maps, leaf-collections-first ordering, CSV normalization, the CDN asset sweep across CSV/rich text/HTML/CSS/OG, the component-inventory-before-HTML step, the section-pattern analysis prompt, redirect/URL-map cutover, "don't reorganize a live Webflow CMS to ease export", and the validation checklist.How
skills/sanity-migration/references/webflow.mdwith a distilled, self-contained reference (~149 lines), dropping the required "fetch this URL" block and folding its intent (read the guidance, align on decisions, plan intoMIGRATION.md) into a brief lead-in._weakplaceholders resolved in a second pass — the leaf-first ordering is retained alongside it.🤖 Generated with Claude Code
https://claude.ai/code/session_01EwCiGS4nB5F1Benh3jYTun