Skip to content

Conversation

@VidhyaDivakar
Copy link

@VidhyaDivakar VidhyaDivakar commented Jan 23, 2026

  • [Yes ] I have read and followed the Contributing docs, especially the "First-time contributing?" section.
  • [Yes ] This PR has content that I did not fully write myself.
  • [ Yes] I have the experience and knowledge necessary to understand, review, and validate all content in this PR.1

@AndrejKiri
Following are the changes made as per our discussion in the https://opentelemetry.io/community/end-user/

  1. Added a CTA "Review issues" (HTML based, Hugo Friendly) buttom linking to a list of projects in OTel org.
  2. Turned this link Join CNCF Slack. to a (HTML based, Hugo Friendly) with Slack logo as suggested.
  3. Added the file slack-logo.svg in static/img/slack-logo.svg for use with the button in the End User Resources Page.

I have created a Hugo site and tested on this in my local. Please find below the preview of the same.

image image

Please review and let me know.

Thanks
Vidhya

Footnotes

  1. Yes, I can answer maintainer questions about the content of this PR, without using AI.

@VidhyaDivakar
Copy link
Author

VidhyaDivakar commented Jan 23, 2026

@AndrejKiri, I did some testing and previewed in the deploy preview site. Please review the button and let me know. I can see a (Merging is blocked icon, Kindly check that too.)

-Thanks,
Vidhya

Copy link
Contributor

@AndrejKiri AndrejKiri left a comment

Choose a reason for hiding this comment

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

Thank you so much for accepting the feedback and creating this PR Vidhya! 🙌

Left a few comments. Looking forward to seeing how it renders in the preview instance.


<a href="https://github.com/orgs/open-telemetry/projects" target="_blank" rel="noopener noreferrer"><button style="background-color:#1e5eb8;color:#f0f6fc;border:none;padding:6px 14px;border-radius:6px;font-weight:600;cursor:pointer;">Review Issues</button></a>

<!--➡️ TODO: Add a CTA "Review issues" linking to a list of projects in OTel org ) -->
Copy link
Contributor

Choose a reason for hiding this comment

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

We can remove this.

Suggested change
<!--➡️ TODO: Add a CTA "Review issues" linking to a list of projects in OTel org ) -->

what we’re looking for. Are you the first person who bumped into this problem?
File an issue in one of our repositories.

<a href="https://github.com/orgs/open-telemetry/projects" target="_blank" rel="noopener noreferrer"><button style="background-color:#1e5eb8;color:#f0f6fc;border:none;padding:6px 14px;border-radius:6px;font-weight:600;cursor:pointer;">Review Issues</button></a>
Copy link
Contributor

Choose a reason for hiding this comment

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

To my surprise, previous link opens the recent projects someone visited what is not always useful. We want to open all the projects. Here is a link to do that:

Suggested change
<a href="https://github.com/orgs/open-telemetry/projects" target="_blank" rel="noopener noreferrer"><button style="background-color:#1e5eb8;color:#f0f6fc;border:none;padding:6px 14px;border-radius:6px;font-weight:600;cursor:pointer;">Review Issues</button></a>
<a href="https://github.com/orgs/open-telemetry/projects?query=is%3Aopen" target="_blank" rel="noopener noreferrer"><button style="background-color:#1e5eb8;color:#f0f6fc;border:none;padding:6px 14px;border-radius:6px;font-weight:600;cursor:pointer;">Review Issues</button></a>

[Join CNCF Slack.](https://communityinviter.com/apps/cloud-native/cncf)
<a href="https://communityinviter.com/apps/cloud-native/cncf" target="_blank" rel="noopener noreferrer" style="text-decoration:none;display:inline-block;"> <button style="background-color:#1e5eb8;color:#f0f6fc;border:none;padding:6px 12px;border-radius:6px;font-weight:600;cursor:pointer;display:flex;align-items:center;gap:8px;"><span style="order:1;">Join CNCF Slack</span><img src="/img/slack-logo.svg" alt="Slack" style="width:16px;height:16px;order:2;"></button></a>

<!--➡️ TODO: Turn link to a button -->
Copy link
Contributor

Choose a reason for hiding this comment

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

We can remove this as well.

Suggested change
<!--➡️ TODO: Turn link to a button -->

@AndrejKiri
Copy link
Contributor

Btw, when you mentioned that the buttons are Hugo-friendly, I googled that a bit and found also this approach:

{{< button href="https://github.com/" variant="primary" size="large" >}} Learn more {{< /button >}}

Not sure if this would work and I am not suggesting to necessarily use this. Up to you. Just wanted to share something I bumped into. Just FYI that something like that might exist.

@chalin chalin force-pushed the End-user-resources-Webpage-update branch from c4854d5 to ffa9dd5 Compare January 29, 2026 10:49
@chalin
Copy link
Contributor

chalin commented Jan 29, 2026

/fix:all

@otelbot-docs otelbot-docs bot requested a review from a team January 29, 2026 10:49
@otelbot-docs
Copy link
Contributor

otelbot-docs bot commented Jan 29, 2026

fix:all applied successfully in run 21475267035.

Copy link
Contributor

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Here's a cleaner way to add CTA buttons:

Sorry, I would have added suggested edits where, but it got a little too much, so I submitted a separate PR>

@chalin
Copy link
Contributor

chalin commented Jan 29, 2026

Btw, for future reference, use:

as the link to have folks sign up to Slack, not https://communityinviter.com/apps/cloud-native/cncf.

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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants