Skip to content

Migrate /contact-us, /thank-you/* and /book-demo from 11ty to Nuxt - #5537

Open
ZJvandeWeg wants to merge 3 commits into
mainfrom
zj-nuxt-contact-book-demo
Open

Migrate /contact-us, /thank-you/* and /book-demo from 11ty to Nuxt#5537
ZJvandeWeg wants to merge 3 commits into
mainfrom
zj-nuxt-contact-book-demo

Conversation

@ZJvandeWeg

@ZJvandeWeg ZJvandeWeg commented Aug 7, 2026

Copy link
Copy Markdown
Member

Ports the shared mql-contact layout, HubSpot form, and a new meetings embed component, then flips CtaContactUs/CtaBookDemo's external prop to false now that both routes are served by Nuxt directly.

Also migrates /thank-you to Nuxt (4 route variants, new stories/webinar/blog preview components, stories/webinars content collections sourced from src/), and inverts the dev proxy's routing list in legacy.ts: it now lists what's still on 11ty instead of what's migrated, so the list shrinks toward empty as pages move over.

@ZJvandeWeg
ZJvandeWeg requested a review from Yndira-E August 7, 2026 18:00
@netlify

netlify Bot commented Aug 7, 2026

Copy link
Copy Markdown

Deploy Preview for flowforge-website ready!

Name Link
🔨 Latest commit efc82bf
🔍 Latest deploy log https://app.netlify.com/projects/flowforge-website/deploys/6a7c83618f5e0c00080558c4
😎 Deploy Preview https://deploy-preview-5537--flowforge-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 60 (🔴 down 3 from production)
Accessibility: 96 (no change from production)
Best Practices: 100 (no change from production)
SEO: 85 (🔴 down 7 from production)
PWA: -
View the detailed breakdown and full score reports

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

Comment thread nuxt/content.config.ts

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@ZJvandeWeg I'm not sure if this was meant to be included in this PR, the title and description only talk about contact us and book a demo. Was this intentional?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, intentional — the title/description have been updated to reflect that this PR also migrates /thank-you/*.

Comment thread nuxt/pages/thank-you/download-platform-overview.vue
@Yndira-E
Yndira-E requested review from KristopherLeads and removed request for KristopherLeads August 11, 2026 14:23
Ports the shared mql-contact layout, HubSpot form, and a new meetings
embed component, then flips CtaContactUs/CtaBookDemo's `external` prop
to false now that both routes are served by Nuxt directly.
… to Nuxt

legacy.ts now lists what's still on 11ty instead of what's migrated, so the
list shrinks toward empty as pages move over. /thank-you is fully ported:
4 route variants, new stories/webinar/blog preview components, and stories/
webinars content collections sourced in place from src/.
Comment thread nuxt/server/middleware/legacy.ts Outdated
@Yndira-E

Yndira-E commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

@ZJvandeWeg on local, images that are still served by 11ty aren't loading, and the terminal is flooded with repeated [Vue warn]: Invalid prop: type check failed for prop "value". Expected Object, got Undefined at <ContentRenderer> warnings on basically every page (except for the blog and changelog).

Looks like it's a side effect of inverting the proxy split in legacy.ts.

@ZJvandeWeg ZJvandeWeg changed the title Migrate /contact-us and /book-demo from 11ty to Nuxt Migrate /contact-us, /thank-you/* and /book-demo from 11ty to Nuxt Aug 11, 2026
Co-authored-by: Yndira Escobar <129537638+Yndira-E@users.noreply.github.com>

@Yndira-E Yndira-E left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey @ZJvandeWeg Following up on this comment.

The local dev experience is broken now. Images served by 11ty don't load locally on most pages, with repeated Vue warnings in the terminal. Needs to be fixed before I can approve/merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants