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(breadcrumbs): locale in home url #145

Closed jbmoelker closed 4 weeks ago

jbmoelker commented 1 month ago

Changes

First breadcrumb linking to home page was missing the locale, causing unneeded redirects. This PR adds it.

Associated issue

N/A

How to test

Open preview link
Navigate to any nested page
Tap on home breadcrumb
Verify that it directly navigates to the home page in the right locale without redirects

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)
~~I have made updated relevant documentation files (in project README, docs/, etc)~~
~~I have added a decision log entry if the change affects the architecture or changes a significant technology~~
[x] I have notified a reviewer

cloudflare-pages[bot] commented 1 month ago

Deploying head-start with Cloudflare Pages

Latest commit:	`dbcb129`
Status:	✅ Deploy successful!
Preview URL:	https://d3520f8c.head-start.pages.dev
Branch Preview URL:	https://fix-breadcrumbs-home-url.head-start.pages.dev