Skip to content

fix: links of breadcrumbs do not respect BASE_URL#210

Merged
eyeix merged 4 commits intoeyeix:mainfrom
nifr:fix/breadcrums-do-not-use-base-url
Apr 3, 2025
Merged

fix: links of breadcrumbs do not respect BASE_URL#210
eyeix merged 4 commits intoeyeix:mainfrom
nifr:fix/breadcrums-do-not-use-base-url

Conversation

@nifr
Copy link
Contributor

@nifr nifr commented Mar 31, 2025

I deployed both Meilisearch and meilisearch-ui on a single instance and with a BASE_PATH of /ui/.

I thought my webserver routing was incorrect after I experienced some 404s when I clicked on the breadcrumbs.
A closer look revealed that the links generated for the breadcrumbs are not correct.

This PR adds BASE_URL as a prefix to the href of the breadcrumbs.

@vercel
Copy link

vercel bot commented Mar 31, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
meilisearch-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 3, 2025 1:41pm

baseUrl empty by default when no BASE_PATH, give another slash value as default at home link.
@eyeix eyeix merged commit cb8b5ef into eyeix:main Apr 3, 2025
2 checks passed
@eyeix
Copy link
Owner

eyeix commented Apr 3, 2025

@nifr Thx!

@eyeix eyeix mentioned this pull request Apr 3, 2025
This was referenced Apr 24, 2025
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