voorhoede / head-start

Base setup on top of headless services to help you quickly start a new website

ISC License

3 stars 0 forks source link

fix: Preview mode routing #127

Closed jbmoelker closed 5 months ago

jbmoelker commented 5 months ago

Changes

Errors occurred in preview mode caused by incorrect API route URLs and the new nested pages route ([...path]). This PR resolves those by:

Adds trailing slashes to preview API routes in templates and docs, as they are now required.
Handle page route mismatch, caused by open Astro issue.

Associated issue

N/A

How to test

Open preview link
Enter preview secret
Verify you enter preview mode
Verify home page(s) render
Verify generic page(s) render
Exit preview mode
Verify that works too

Checklist

[x] I have performed a self-review of my own code
[x] I have made sure that my PR is easy to review (not too big, includes comments)
[x] I have made updated relevant documentation files (in project README, docs/, etc) 👉 inline comments
~~I have added a decision log entry if the change affects the architecture or changes a significant technology~~
~~I have notified a reviewer~~

cloudflare-pages[bot] commented 5 months ago

Deploying with Cloudflare Pages

Latest commit:	`6699deb`
Status:	✅ Deploy successful!
Preview URL:	https://4b61b1bd.head-start.pages.dev
Branch Preview URL:	https://fix-preview-mode-routing.head-start.pages.dev