Skip to content

fix(collector): show per-signal stability levels and remove generic fallback description#427

Open
aabhinavvvvvvv wants to merge 2 commits into
open-telemetry:mainfrom
aabhinavvvvvvv:fix/375-collector-list-stability-details
Open

fix(collector): show per-signal stability levels and remove generic fallback description#427
aabhinavvvvvvv wants to merge 2 commits into
open-telemetry:mainfrom
aabhinavvvvvvv:fix/375-collector-list-stability-details

Conversation

@aabhinavvvvvvv
Copy link
Copy Markdown
Contributor

@aabhinavvvvvvv aabhinavvvvvvv commented May 10, 2026

Summary

Closes #375

  • Replace the single stability badge on collector list cards with a full breakdown of all stability levels and their associated signals (e.g., [stable] logs, metrics, traces / [alpha] profiles)
  • Remove the generic "Browse technical details and configuration options for this component." fallback text — cards with no description now render blank instead

Screenshots

Before:
image

After:
image

@aabhinavvvvvvv aabhinavvvvvvv requested review from a team as code owners May 10, 2026 17:04
@netlify
Copy link
Copy Markdown

netlify Bot commented May 10, 2026

Deploy Preview for otel-ecosystem-explorer ready!

Name Link
🔨 Latest commit 1ed0dcc
🔍 Latest deploy log https://app.netlify.com/projects/otel-ecosystem-explorer/deploys/6a0314d55e9ab7000866eb3c
😎 Deploy Preview https://deploy-preview-427--otel-ecosystem-explorer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@aabhinavvvvvvv aabhinavvvvvvv force-pushed the fix/375-collector-list-stability-details branch 3 times, most recently from 6ed6f7f to 0703dd2 Compare May 10, 2026 17:29
@vitorvasc vitorvasc requested a review from Copilot May 11, 2026 08:47
@vitorvasc
Copy link
Copy Markdown
Member

Hi @aabhinavvvvvvv! Could you add some screenshot examples to the description showing the before and after? 🙌🏼

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Collector components list UI to better reflect per-signal stability metadata and to avoid showing a generic description when component descriptions are missing.

Changes:

  • Only render a component description paragraph when comp.description is present (removes the generic fallback text).
  • Replace the single stability badge on list cards with a per-stability-level breakdown showing associated signals.

Comment thread ecosystem-explorer/src/features/collector/collector-page.tsx Outdated
Comment thread ecosystem-explorer/src/features/collector/collector-page.tsx
@aabhinavvvvvvv aabhinavvvvvvv force-pushed the fix/375-collector-list-stability-details branch from 0703dd2 to ca17d5c Compare May 11, 2026 11:19
@aabhinavvvvvvv aabhinavvvvvvv force-pushed the fix/375-collector-list-stability-details branch from ca17d5c to 5739341 Compare May 11, 2026 11:21
@aabhinavvvvvvv
Copy link
Copy Markdown
Contributor Author

@vitorvasc I've added screenshots and resolved copilot's comments

@vitorvasc vitorvasc self-assigned this May 13, 2026
@vitorvasc
Copy link
Copy Markdown
Member

Thanks @aabhinavvvvvvv. I'll get back to it as soon as possible.

@vitorvasc
Copy link
Copy Markdown
Member

Hi @aabhinavvvvvvv, thanks for your patience!

There are two things I'd like us to address before merging:

  1. Multi-stability components will render two (or more) rows on the card, and things could messy in this scenario. It's worth testing alternatives for how we can show that information before merging, such as using tooltips or collapsible elements.
  2. Signal names such as logs_to_metrics or traces_to_traces seem too "raw" for a card surface, can we normalize these to human-readable names? I'd say that could even mean stripping the _to_* suffix, but I'll ask for a second opinion from a Collector maintainer here. @mx-psi, any thoughts on this?

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.

Collector list page updates

3 participants