Skip to content

Handle linkchecker result discovered in this week's check - #975

Merged
Scotchester merged 2 commits into
mainfrom
chore/fix-linkchecker-results
Sep 8, 2026
Merged

Handle linkchecker result discovered in this week's check#975
Scotchester merged 2 commits into
mainfrom
chore/fix-linkchecker-results

Conversation

@Scotchester

Copy link
Copy Markdown
Member

Closes #973


At some point between testing this a week or two ago and today, the Google Tag Manager URL started returning a 500 if something tries to access it directly. (It still results in a 200 when requested by the actual site.) This resulted in a linkchecker finding during yesterday's weekly check.

This PR handles it by just ignoring all URLs with googletagmanager in them.

It also updates an extant link to /contact, which I discovered that when testing this fix. Eleventy doesn't inherently support our Netlify _redirects file, so it was 404ing locally.

@Scotchester
Scotchester requested a review from a team as a code owner September 8, 2026 20:11
@netlify

netlify Bot commented Sep 8, 2026

Copy link
Copy Markdown

Deploy Preview for cal-itp-mobility-marketplace ready!

Name Link
🔨 Latest commit 1b011c9
🔍 Latest deploy log https://app.netlify.com/projects/cal-itp-mobility-marketplace/deploys/6aa06be4df2a640008a4b55f
😎 Deploy Preview https://deploy-preview-975--cal-itp-mobility-marketplace.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.

@jgravois jgravois left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

:bowtie:

@Scotchester
Scotchester merged commit e816457 into main Sep 8, 2026
7 checks passed
@Scotchester
Scotchester deleted the chore/fix-linkchecker-results branch September 8, 2026 20:36
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 found

2 participants