Skip to content

Link certified node cards to their documentation pages - #5522

Open
dimitrieh wants to merge 1 commit into
mainfrom
link-certified-node-cards
Open

Link certified node cards to their documentation pages#5522
dimitrieh wants to merge 1 commit into
mainfrom
link-certified-node-cards

Conversation

@dimitrieh

Copy link
Copy Markdown
Contributor

Description

The connector cards in the Certified Nodes showcase look interactive but are plain divs, so clicking one does nothing. This links each card to that node's documentation page, using the same targets the header's "By Integration" menu already uses.

Both places the showcase appears are covered: /pricing/ (hand-curated, nuxt/content/certified-nodes.yml) and /integrations/ (live catalogue). The /node-red/ certified grid and the in-tutorial callout already linked.

Also in scope, same defect class:

  • The /integrations/ grid card read node.collection where a catalogue entry only carries collections, so its certified fallback never resolved.
  • Documentation paths taken from a catalogue url now get a trailing slash, so a click does not go through a redirect.

Related Issue(s)

None.

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)

The connector cards in the Certified Nodes showcase looked interactive but
were plain divs, on /pricing/ and on /integrations/ alike, so clicking one
did nothing. Both showcases now link each card to that node's documentation
page, using the same targets the header's By Integration menu already uses.

Also fixes the certified fallback in the /integrations/ grid card, which
read node.collection where a catalogue entry only carries collections, and
normalises the trailing slash on documentation paths taken from a catalogue
url so a click does not go through a redirect.
@dimitrieh

Copy link
Copy Markdown
Contributor Author

@dimitrieh
dimitrieh requested review from Yndira-E and n-lark August 6, 2026 09:49
@netlify

netlify Bot commented Aug 6, 2026

Copy link
Copy Markdown

Deploy Preview for flowforge-website ready!

Name Link
🔨 Latest commit cb8f32b
🔍 Latest deploy log https://app.netlify.com/projects/flowforge-website/deploys/6a7453d6a48aaa00080dd844
😎 Deploy Preview https://deploy-preview-5522--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: 65 (🟢 up 1 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.

:to="node.url"
class="group flex items-start gap-3 rounded-xl border border-gray-200 bg-white p-3 transition hover:border-gray-300 hover:shadow-sm hover:no-underline"
@click="captureNodeClick(node.name, b)"
>

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.

<NuxtLink> in CertifiedNodes.vue points at 11ty routes that aren't Nuxt routes, so clicking a /pricing/ card 404s in prod build. Works locally only via the dev proxy. Once that is fixed lmk and I'll re review.

Image

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