Skip to content

fix: add sidebar icons for analytics pages#165

Closed
Kinfe123 wants to merge 1 commit intomainfrom
codex/sidebar-icons
Closed

fix: add sidebar icons for analytics pages#165
Kinfe123 wants to merge 1 commit intomainfrom
codex/sidebar-icons

Conversation

@Kinfe123
Copy link
Copy Markdown
Member

@Kinfe123 Kinfe123 commented May 8, 2026

Summary\n- register the existing Analytics frontmatter icon key with lucide's ChartNoAxesColumn icon\n- register the existing Observability frontmatter icon key with lucide's Activity icon\n\n## Verification\n- pnpm format:check\n- pnpm exec next build from website/


Summary by cubic

Adds sidebar icons for Analytics and Observability docs by mapping existing frontmatter keys to lucide-react icons. Registers chart-no-axes-column → ChartNoAxesColumn and activity → Activity so the docs sidebar shows the correct icons.

Written for commit e4a60d7. Summary will update on new commits.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs-website Ready Ready Preview, Comment May 8, 2026 0:25am

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 8, 2026

Open in StackBlitz

@farming-labs/astro

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/astro@165

@farming-labs/astro-theme

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/astro-theme@165

@farming-labs/docs

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/docs@165

@farming-labs/theme

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/theme@165

@farming-labs/next

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/next@165

@farming-labs/nuxt

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/nuxt@165

@farming-labs/nuxt-theme

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/nuxt-theme@165

@farming-labs/svelte

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/svelte@165

@farming-labs/svelte-theme

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/svelte-theme@165

@farming-labs/tanstack-start

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/tanstack-start@165

commit: e4a60d7

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

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.

1 participant