Skip to content

fix:Refactor Collector component URLs to distribution/name structure#420

Open
omthorat2004 wants to merge 1 commit into
open-telemetry:mainfrom
omthorat2004:feat/collector-url-restructure
Open

fix:Refactor Collector component URLs to distribution/name structure#420
omthorat2004 wants to merge 1 commit into
open-telemetry:mainfrom
omthorat2004:feat/collector-url-restructure

Conversation

@omthorat2004
Copy link
Copy Markdown

fixes #368
This PR refactors Collector component routing and ID structure.

Changes

  • Component ID format changed from:
    {distribution}-{type}-{name}
    → {distribution}-{name}

  • Updated frontend routing:
    /collector/components/:version/:id
    → /collector/components/:distribution/:name

  • Version moved from path to query param:
    ?version=0.150.0

  • Updated data loading and transformation logic

  • Updated tests and assertions to match new ID format

Migration

Regenerated collector data with updated ID format. No manual migration required.

@omthorat2004 omthorat2004 requested review from a team as code owners May 10, 2026 13:53
@netlify
Copy link
Copy Markdown

netlify Bot commented May 10, 2026

Deploy Preview for otel-ecosystem-explorer canceled.

Name Link
🔨 Latest commit c2b3157
🔍 Latest deploy log https://app.netlify.com/projects/otel-ecosystem-explorer/deploys/6a01a2d43905860008fc3887

@omthorat2004 omthorat2004 force-pushed the feat/collector-url-restructure branch from a123b1c to ad4be3f Compare May 10, 2026 13:55
@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented May 11, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: omthorat2004 / name: Om Thorat (c2b3157)

@vitorvasc
Copy link
Copy Markdown
Member

Hi @omthorat2004, could you sign the CLA?

@vitorvasc vitorvasc added add-screenshots missing:cla The CLA has not been signed by the PR author, therefore the PR can not be merged. and removed add-screenshots labels May 11, 2026
@omthorat2004
Copy link
Copy Markdown
Author

omthorat2004 commented May 11, 2026

@vitorvasc CLA is done!
Thanks.
I was also thinking to add add-screenshots label as these are ui changes.

@omthorat2004 omthorat2004 force-pushed the feat/collector-url-restructure branch 2 times, most recently from 78e5afc to 1838b44 Compare May 11, 2026 09:30
@omthorat2004 omthorat2004 force-pushed the feat/collector-url-restructure branch from 1838b44 to c2b3157 Compare May 11, 2026 09:35
@jaydeluca
Copy link
Copy Markdown
Member

@omthorat2004 could you update your branch with main so the diff is clean?

I was also thinking to add add-screenshots label as these are ui changes.

Feel free to throw screenshots from testing into the PR description or comments, we always welcome that to help provide context on reviews 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

missing:cla The CLA has not been signed by the PR author, therefore the PR can not be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Restructure Collector Component URLs

3 participants