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

[infra] Better support for RSS and blog pages #6002

Merged

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Jan 21, 2025

Preview:

RSS feeds:

/cc @dmathieu

@chalin chalin added CI/infra CI & infrastructure blog labels Jan 21, 2025
@chalin chalin requested a review from a team as a code owner January 21, 2025 12:59
@opentelemetrybot opentelemetrybot requested review from a team and krol3 and removed request for a team January 21, 2025 12:59
@dmathieu
Copy link
Member

How about adding link tags with the feed's address?

@chalin
Copy link
Contributor Author

chalin commented Jan 21, 2025

How about adding link tags with the feed's address?

The link tag is there, e.g.: <link>https://deploy-preview-6002--opentelemetry.netlify.app/</link> for /index.xml. From the blog feed:

<rss xmlns:atom="http://www.w3.org/2005/Atom" version="2.0">
<channel>
<title>Blog on OpenTelemetry</title>
<link>https://deploy-preview-6002--opentelemetry.netlify.app/blog/</link>
<description>Recent content in Blog on OpenTelemetry</description>
...

@chalin
Copy link
Contributor Author

chalin commented Jan 21, 2025

/cc @austinlparker

@dmathieu
Copy link
Member

The per-language feeds don't seem to be linked when I visit a page such as https://deploy-preview-6002--opentelemetry.netlify.app/pt/blog/ though.

@chalin
Copy link
Contributor Author

chalin commented Jan 21, 2025

The per-language feeds don't seem to be linked when I visit a page such as https://deploy-preview-6002--opentelemetry.netlify.app/pt/blog/ though.

Right. I'm not quite satisfied with the way the non-en blogs are setup, and so they are likely to be reworked. For that reason I had disabled the blog RSS feeds for non-en locales. I guess that I can leave the active in the meantime. Let me do that.

@opentelemetrybot opentelemetrybot requested review from a team January 21, 2025 14:04
@opentelemetrybot opentelemetrybot requested a review from a team January 21, 2025 14:04
@chalin chalin merged commit c0a5eea into open-telemetry:main Jan 21, 2025
17 checks passed
@chalin chalin deleted the chalin-im-blog-wo-redirect-2025-01-21 branch January 21, 2025 14:13
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.

Create 2025 blog section and adjust redirects
3 participants