Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ecosystem section #2185

Merged
merged 11 commits into from
Jan 24, 2023
Merged

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Jan 20, 2023

@svrnm @cartermp @austinlparker et al., here's a proposal I've been juggling with ever since we've been wanting to make the demo more prominent.

This PR:

  • Adds a website top-level section /ecosystem
  • Replaces the Registry top-nav entry with a link to Ecosystem
  • Adds (placeholder) subpages to Ecosystem (see note below as to why these are placeholder pages):
    • Demo
    • Registry
    • Community
  • Removes Community from the top-nav

For this PR, the Ecosystem subpages are placeholder pages only (i.e., just links) because I didn't want to move the corresponding sections. We can either leave them as placeholder pages, or relocate them (while introducing appropriate aliases).

If you like the idea overall, we could also remove the placeholder pages to the demo that are found under Community and the Getting started sections (but I'd handle that separately).

Thoughts?

Preview: https://deploy-preview-2185--opentelemetry.netlify.app/ecosystem/


Screenshot

image

@chalin chalin added the enhancement New feature or request label Jan 20, 2023
@chalin chalin requested a review from a team January 20, 2023 18:59
@svrnm
Copy link
Member

svrnm commented Jan 23, 2023

I am overall in favor of this. We can also have a page on vendors/integrators/adopters as another section. I am just wondering if people will know what to expect when clicking on "Ecosystem"

@open-telemetry/end-user-wg asking you for your opinion as well

content/en/ecosystem/_index.md Outdated Show resolved Hide resolved
@austinlparker
Copy link
Member

I'd agree with putting the vendor stuff under here as well, although I'm not sure about 'community'? I guess it just feels a bit out of place with the rest of it, but I don't really have an alternative. I do like the demo page being there.

Copy link
Contributor

@cartermp cartermp left a comment

Choose a reason for hiding this comment

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

This sparks joy.

@chalin
Copy link
Contributor Author

chalin commented Jan 23, 2023

... I'm not sure about 'community'? I guess it just feels a bit out of place with the rest of it, ...

I can certainly put Community back where it was. I agree that it feels a bit odd. WDYT @svrnm @cartermp et al.?

@svrnm
Copy link
Member

svrnm commented Jan 24, 2023

... I'm not sure about 'community'? I guess it just feels a bit out of place with the rest of it, ...

I can certainly put Community back where it was. I agree that it feels a bit odd. WDYT @svrnm @cartermp et al.?

would the following work:

  • community is more "human-focused" and holds details on contributions, end-user working groups, the vision&mission&roadmap
  • ecosystem is more "tech-focused" and holds details on the demo, vendors, adopters, integrators, the registry, etc.

@chalin chalin force-pushed the chalin-im-ecosys-2023-01-20 branch from 7e4d11b to ad0ecff Compare January 24, 2023 18:12
@musingvirtual
Copy link
Contributor

Strong second to Austin's point - I don't think I would know to look under "ecosystem" for community. Thanks for putting it back in the top nav bar.

@chalin chalin requested review from a team January 24, 2023 18:45
@chalin
Copy link
Contributor Author

chalin commented Jan 24, 2023

Hi all.

  • Community top-nav entry is back
  • Added to Ecosystem:
    • Integrations
    • Vendors

PTAL

@chalin
Copy link
Contributor Author

chalin commented Jan 24, 2023

I also move the actual demo page from Community to Ecosystem.

@chalin
Copy link
Contributor Author

chalin commented Jan 24, 2023

/cc @open-telemetry/demo-approvers

@chalin
Copy link
Contributor Author

chalin commented Jan 24, 2023

FYI, I just adjusted the CODEOWNERS file so that it refers to the new demo page location.

@chalin chalin mentioned this pull request Jan 24, 2023
27 tasks
@svrnm
Copy link
Member

svrnm commented Jan 24, 2023

Great stuff!

@cartermp cartermp merged commit 9fbbc23 into open-telemetry:main Jan 24, 2023
@chalin chalin deleted the chalin-im-ecosys-2023-01-20 branch January 25, 2023 11:41
@chalin chalin mentioned this pull request Jan 25, 2023
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants