Skip to content

[codex] add Storybook and Grafana to project list#607

Closed
Sean-Kenneth-Doherty wants to merge 1 commit into
firstcontributions:mainfrom
Sean-Kenneth-Doherty:codex/add-storybook-grafana-projects
Closed

[codex] add Storybook and Grafana to project list#607
Sean-Kenneth-Doherty wants to merge 1 commit into
firstcontributions:mainfrom
Sean-Kenneth-Doherty:codex/add-storybook-grafana-projects

Conversation

@Sean-Kenneth-Doherty
Copy link
Copy Markdown

Summary

  • Adds Storybook to the project list with a GitHub avatar, contribution link, tags, and live beginner issue loading.
  • Adds Grafana to the project list with a GitHub avatar, contribution link, tags, and live beginner issue loading.
  • Contributes to Add projects to project list #1.

Why

Both projects are active, unarchived repositories with open good first issue items, making them useful additions for first-time contributors browsing the project list.

Validation

  • pnpm build
  • git diff --check
  • Verified neither project is already present in src/data/projects.js and no open PR mentions Storybook or Grafana.
  • Verified both /contribute pages return HTTP 200 and both avatar URLs return image content.
  • Verified the built index contains both cards and renders three issue links for each project.

@Sean-Kenneth-Doherty
Copy link
Copy Markdown
Author

I rechecked this in an isolated clone before taking it out of draft:

  • pnpm install --frozen-lockfile
  • pnpm build
  • Storybook and Grafana cards are present in dist/index.html
  • Both projects still have current good first issue results and their /contribute pages/avatar URLs respond successfully

Keeping this draft for now because the built issue cards still include the literal issue-label conditional from the separate IssueList.astro class binding bug. That fix is now in #610; this PR should either wait for #610 to land or be rebased after it.

@Sean-Kenneth-Doherty Sean-Kenneth-Doherty marked this pull request as ready for review May 16, 2026 23:53
@Sean-Kenneth-Doherty
Copy link
Copy Markdown
Author

Withdrawing to reduce review noise from the large batch of small project-list PRs. This also depended on the issue-label rendering fix landing first, so it is better not to leave it open as a separate review item.

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