Skip to content

Fix broken external links in Footer, replace <Link> with <a>#1071

Merged
capfei merged 3 commits into
clearlydefined:masterfrom
julianladisch:footer-Link
Mar 20, 2026
Merged

Fix broken external links in Footer, replace <Link> with <a>#1071
capfei merged 3 commits into
clearlydefined:masterfrom
julianladisch:footer-Link

Conversation

@julianladisch

Copy link
Copy Markdown
Contributor

<Link> doesn't support external links in react-router versions before 6.8.0: https://github.com/remix-run/react-router/blob/main/CHANGELOG.md#v680

We use react-router version 4.3.1 resulting in broken links to Terms, Privacy, and Notices.

This gets fixed by replacing <Link> with <a>.

Fixes #995.

`<Link>` doesn't support external links in react-router versions before 6.8.0:
https://github.com/remix-run/react-router/blob/main/CHANGELOG.md#v680

We use react-router version 4.3.1 resulting in broken links to
Terms, Privacy, and Notices.

This gets fixed by replacing `<Link>` with `<a>`.

Fixes clearlydefined#995.
@netlify

netlify Bot commented Jun 28, 2025

Copy link
Copy Markdown

Deploy Preview for blissful-goodall-fa23f6 ready!

Name Link
🔨 Latest commit cd29781
🔍 Latest deploy log https://app.netlify.com/projects/blissful-goodall-fa23f6/deploys/686023e1b1f7be0008b8b418
😎 Deploy Preview https://deploy-preview-1071--blissful-goodall-fa23f6.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@netlify

netlify Bot commented Feb 6, 2026

Copy link
Copy Markdown

Deploy Preview for blissful-goodall-fa23f6 failed.

Name Link
🔨 Latest commit c2620ea
🔍 Latest deploy log https://app.netlify.com/projects/blissful-goodall-fa23f6/deploys/69bda00f6eeaaa00088d3f50

@capfei capfei merged commit c9a8bb5 into clearlydefined:master Mar 20, 2026
2 of 6 checks passed
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.

Broken links at footer of page (Terms | Privacy | Notices)

2 participants