[PR #51] [MERGED] Improved RSS Reader Support #97

New issue

Closed

opened 2026-02-25 23:39:54 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 23:39:54 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/HaschekSolutions/opentrashmail/pull/51
Author: @adamklaff
Created: 8/22/2023
Status: ✅ Merged
Merged: 8/22/2023
Merged by: @geek-at

Base: master ← Head: feedreader

📝 Commits (3)

3e443d4 Remove Dockerfile label
200dd2d 4 digit year, add guid, remove 5 article limit
c05a9c3 Update Dockerfile

📊 Changes

2 files changed (+42 additions, -43 deletions)

View changed files

📝 docker/Dockerfile (+1 -1)
📝 web/rss.php (+41 -42)

📄 Description

I couldn't get my RSS feed to appear in my feed reader and after checking an RSS feed validator, changed the following:

Added 4 digit year in the Pub Date
Add a GUID element that links back to the message ID. Ideally there could be a permalink to render HTML output of the message but I didn't see how this was easily implemented.
Removed the 5 article limit for RSS rendering. I don't know why this was added (timeouts?) but I need to get all articled into the reader.

Looks like there may have been some linting changes

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/HaschekSolutions/opentrashmail/pull/51 **Author:** [@adamklaff](https://github.com/adamklaff) **Created:** 8/22/2023 **Status:** ✅ Merged **Merged:** 8/22/2023 **Merged by:** [@geek-at](https://github.com/geek-at) **Base:** `master` ← **Head:** `feedreader` --- ### 📝 Commits (3) - [`3e443d4`](https://github.com/HaschekSolutions/opentrashmail/commit/3e443d4875bbc37723872b2014b944ed31a29577) Remove Dockerfile label - [`200dd2d`](https://github.com/HaschekSolutions/opentrashmail/commit/200dd2d8c6027aba0324484a25feb64bc0722ad0) 4 digit year, add guid, remove 5 article limit - [`c05a9c3`](https://github.com/HaschekSolutions/opentrashmail/commit/c05a9c329f10214129cf6ef64d023fe98cd13f7b) Update Dockerfile ### 📊 Changes **2 files changed** (+42 additions, -43 deletions) <details> <summary>View changed files</summary> 📝 `docker/Dockerfile` (+1 -1) 📝 `web/rss.php` (+41 -42) </details> ### 📄 Description I couldn't get my RSS feed to appear in my feed reader and after checking an RSS feed validator, changed the following: - Added 4 digit year in the Pub Date - Add a GUID element that links back to the message ID. Ideally there could be a permalink to render HTML output of the message but I didn't see how this was easily implemented. - Removed the 5 article limit for RSS rendering. I don't know why this was added (timeouts?) but I need to get all articled into the reader. Looks like there may have been some linting changes --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>