Skip to content

feat(site): Certificate dev links#4390

Merged
ericfennis merged 15 commits into
mainfrom
certifcate-links
May 20, 2026
Merged

feat(site): Certificate dev links#4390
ericfennis merged 15 commits into
mainfrom
certifcate-links

Conversation

@ericfennis

Copy link
Copy Markdown
Member

Closed partnership with Certificates.dev, added certificates dev links to the aside in pages

@ericfennis ericfennis requested a review from Copilot May 20, 2026 06:30
@github-actions github-actions Bot added the 🌍 site Has to do something with the Lucide website label May 20, 2026

Copilot AI left a comment

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.

Pull request overview

Adds a new Certificates.dev partnership promotion to the VitePress docs theme by rendering a framework-specific link/image in the page aside.

Changes:

  • Add a new CertificatesLink Vue component that selects a Certificates.dev URL based on the current docs route.
  • Inject the new component into the VitePress theme layout via the aside-ads-before slot.
  • Update docs head config to include an additional third-party script tag.

Reviewed changes

Copilot reviewed 4 out of 8 changed files in this pull request and generated 2 comments.

File Description
docs/.vitepress/transformPageData.ts Changes the VitePress PageData import (currently introduces a runtime import issue).
docs/.vitepress/theme/index.ts Registers CertificatesLink into the docs theme layout (aside-ads-before).
docs/.vitepress/theme/components/guide/CertificatesLink.vue New aside component rendering a framework-specific Certificates.dev link.
docs/.vitepress/getHeadConfig.ts Adds an additional external script to the global head config.
Comments suppressed due to low confidence (1)

docs/.vitepress/transformPageData.ts:6

  • PageData is only used as a type annotation here, but it’s imported as a runtime value. VitePress exports PageData as a type-only symbol, so this can lead to a runtime import error or unnecessary JS output. Switch back to a type-only import (e.g. import type { PageData } from 'vitepress').
import { PageData } from 'vitepress';
import getStructuredData from './getStructuredData';
import { createGeneralOGImage, createIconOGImage } from './createOGImage';

export async function transformPageData(pageData: PageData) {
  pageData.frontmatter.head ??= [];

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/.vitepress/getHeadConfig.ts
Comment thread docs/.vitepress/theme/components/guide/CertificatesLink.vue Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@ericfennis ericfennis merged commit 7f14e74 into main May 20, 2026
3 checks passed
@ericfennis ericfennis deleted the certifcate-links branch May 20, 2026 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🌍 site Has to do something with the Lucide website

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants