Skip to content

Add BreadcrumbList schema to industry pages - #5773

Open
sumitshinde-84 wants to merge 2 commits into
mainfrom
worktree-breadcrumb-industry-schema
Open

sumitshinde-84 wants to merge 2 commits into
mainfrom
worktree-breadcrumb-industry-schema

Conversation

@sumitshinde-84

Copy link
Copy Markdown
Contributor

Description

Related Issue(s)

Checklist

  • I have read the contribution guidelines
  • I have considered the performance impact of these changes
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
  • For blog PRs, an Art Request has been created (instructions)

Adds a Home > Industries > {industry} BreadcrumbList node to the
shared JSON-LD graph, gated on the "industry" tag so it only applies
to pages under src/industries/.
FAQ answers written as YAML folded scalars end with a trailing
newline, which was being interpolated raw into a JSON string literal
producing invalid JSON-LD (an unescaped control character) on pages
like industries/automotive. Use `| dump | safe` for question/answer
and HowTo name/description/step text so newlines and quotes are
properly escaped.
@sumitshinde-84
sumitshinde-84 force-pushed the worktree-breadcrumb-industry-schema branch from bb7cb16 to 64361f1 Compare September 11, 2026 10:33
@netlify

netlify Bot commented Sep 11, 2026

Copy link
Copy Markdown

Deploy Preview for flowforge-website ready!

Name Link
🔨 Latest commit bb7cb16
🔍 Latest deploy log https://app.netlify.com/projects/flowforge-website/deploys/6aa3d89498b8e3000844a835
😎 Deploy Preview https://deploy-preview-5773--flowforge-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 67 (🟢 up 14 from production)
Accessibility: 96 (no change from production)
Best Practices: 100 (no change from production)
SEO: 92 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

@sumitshinde-84 sumitshinde-84 changed the title Worktree breadcrumb industry schema Add BreadcrumbList schema to industry pages Sep 11, 2026
@netlify

netlify Bot commented Sep 11, 2026

Copy link
Copy Markdown

Deploy Preview for flowforge-website ready!

Name Link
🔨 Latest commit 64361f1
🔍 Latest deploy log https://app.netlify.com/projects/flowforge-website/deploys/6aa3d8f74c350b0008d90b1a
😎 Deploy Preview https://deploy-preview-5773--flowforge-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 76 (🟢 up 23 from production)
Accessibility: 96 (no change from production)
Best Practices: 100 (no change from production)
SEO: 92 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

Comment thread src/_includes/jsonld.njk
"url": "https://www.g2.com/sellers/flowfuse"
}
}
{% if tags and 'industry' in tags %}

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.

@sumitshinde-84 this PR feels void as we're moving away from 11ty in favour of nuxt. Should we close the PR?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes, we're nearing the end of our migration if we merge the separate PRs of #5777

@sumitshinde-84 please see if this needs to be added still after those efforts are merged

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.

3 participants