Skip to content

Added Feature: RSS Feed#415

Merged
ShaneIsrael merged 6 commits intoShaneIsrael:developfrom
smartlizardpy:feature/RSSFeed
Jan 31, 2026
Merged

Added Feature: RSS Feed#415
ShaneIsrael merged 6 commits intoShaneIsrael:developfrom
smartlizardpy:feature/RSSFeed

Conversation

@smartlizardpy
Copy link
Contributor

How does it work: It creates an api endpoint that serves the RSS feed. The endpoint generates the RSS Feed using the RSS Feed template it creates an rss.xml file and serves it to the web. It only displays Public videos on the RSS Feed.

Here are the things that I've added:

  • The RSS API endpoint in api.py
  • Added Settings for the RSS endpoint in settings.js
  • The settings include: Title of site & Description of Site
  • Added constants for the RSS feed settings in constants.py
  • Created the rss.xml file for the template
  • A button in the settings page to open the RSS endpoint

The code works on my machine & the RSS app. I could not run the nginx server on my machine (Arch btw) because of some errors i could not debug. Note there are some errors with greenlet & python 3.14 I had to use python 3.12.8 to run via run_local.sh

@ShaneIsrael
Copy link
Owner

@smartlizardpy for future reference, generally when you have PR reviews you update your code and push to the same branch you PR'd so that you do not have to close the PR and create a new one. Doing that makes it very difficult to track whether you actually fixed the changes because I can't easily see the changes in the diff of where my comments are.

@ShaneIsrael
Copy link
Owner

@smartlizardpy there's a couple merge conflicts. Once those are resolved I'll merge.

@smartlizardpy
Copy link
Contributor Author

@ShaneIsrael Are they on my side?

@ShaneIsrael
Copy link
Owner

Yes, that's why why it the at the bottom of this conversation it says "This branch has conflicts that must be resolved." I merged another branch earlier that also made changes to the Settings.js file, so now those changes conflict with changes you are making. I can't merge your branch in until you update it.

@ShaneIsrael ShaneIsrael merged commit 36f09ae into ShaneIsrael:develop Jan 31, 2026
@smartlizardpy smartlizardpy deleted the feature/RSSFeed branch February 9, 2026 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants