protocolbuffers / protobuf

Protocol Buffers - Google's data interchange format
http://protobuf.dev
Other
65.24k stars 15.45k forks source link

Empty Atom/RSS feed for protobuf.dev/news #12446

Open ejona86 opened 1 year ago

ejona86 commented 1 year ago

Recent release notes keep mentioning Protobuf News as a source of important news. But how are we supposed to learn there are new news items? It seems it is just a bunch of bullets on a page.

In the HTML, Hugo is including alternate reference to an RSS feed, but the feed has no entries. Is that a bug? Or is there an alternate expected form of notification?

Logofile commented 1 year ago

Thanks for raising this issue. I'm looking into how to get Hugo/Docsy to populate the file at protobuf.dev/news/index.xml. It's turning into a bigger effort than it should be.

github-actions[bot] commented 10 months ago

We triage inactive PRs and issues in order to make it easier to find active work. If this issue should remain active or becomes active again, please add a comment.

This issue is labeled inactive because the last activity was over 90 days ago.

ejona86 commented 10 months ago

This is still an issue.

Logofile commented 10 months ago

I agree. This was closed by a bot, but it's still on my to-do list to address. I did do some research into this, but need to pick it back up.

ejona86 commented 8 months ago

Without any way to receive news notifications, News is not a method of announcement. That's a real shame because it is clear effort is being put into them.

Right now they feel more like a notice "on display in the bottom of a locked filing cabinet stuck in a disused lavatory."

Logofile commented 8 months ago

I swear that I've spent several hours trying to fix this! But your wonderful humor has inspired me to give it another go. Wish me luck!

ejona86 commented 6 months ago

The default RSS template finds the news items. But the docsy RSS template doesn't use .Pages or .RegularPages when the parent of the section is Home and instead loops over the entire site's pages and filters by type: where $.Site.RegularPages "Type" $.Section. Unclear why it does that, but it seems it would generally be fine.

But that doesn't work because the news items all have type = "docs". Which... seems strange. It really seems like there should be news/chrony/ and news/release/ sections, and add date to news items to sort them. Since no date is set, removing type = "docs" is not enough, because the pages are in a random order and thus you can't show "the 50 most recent news items."

Backfilling dates is annoying but simple. But I don't know whether y'all would prefer making a new RSS template that uses .Pages (least invasive approach) or removing type = "docs" (probably best long-term, but unclear why that was done originally).

Logofile commented 6 months ago

This is exceedingly helpful information! I was actually talking to one of the creators of the Docsy theme, and even they didn't know about (or at least didn't mention) the type = "docs" issue. I'll play around with removing that and see if that affects the RSS feed in our implementation.

github-actions[bot] commented 3 months ago

We triage inactive PRs and issues in order to make it easier to find active work. If this issue should remain active or becomes active again, please add a comment.

This issue is labeled inactive because the last activity was over 90 days ago.

Logofile commented 3 months ago

This is still on my radar. I will keep this issue open until it is resolved.

github-actions[bot] commented 1 week ago

We triage inactive PRs and issues in order to make it easier to find active work. If this issue should remain active or becomes active again, please add a comment.

This issue is labeled inactive because the last activity was over 90 days ago. This issue will be closed and archived after 14 additional days without activity.