Skip to content
This repository has been archived by the owner on Aug 3, 2024. It is now read-only.

RSS Feeds #1159

Closed
wants to merge 5 commits into from
Closed

RSS Feeds #1159

wants to merge 5 commits into from

Conversation

IMB11
Copy link
Contributor

@IMB11 IMB11 commented May 19, 2023

This pull requests adds RSS, Atom and JSON2 feeds to the frontend.

  • Project Versions RSS
  • User Notifications

Routes

  • /feed/:feed_type/project/:project_id where :feed_type is either rss, atom or json and where :project_id is the ID of the project.
  • /feed/:feed_type/notifications where :feed_type is either rss, atom or json and the user's authentication token is passed via the Authentication header.

@IMB11 IMB11 marked this pull request as ready for review May 21, 2023 13:24
@IMB11
Copy link
Contributor Author

IMB11 commented May 21, 2023

This pull request is ready for review.

@BasiqueEvangelist
Copy link
Contributor

atom feed validator results

Copy link
Contributor

@BasiqueEvangelist BasiqueEvangelist left a comment

Choose a reason for hiding this comment

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

Would also be nice to look at adding categories to feeds and feed items. I haven't really tested how they look in actual readers, but they should be a good way to show users what versions and loaders an item has and filter by them of course, if feed readers support that

setResponseHeader(event, 'Content-Type', 'application/feed+json')
return feed.json1()
default:
setResponseStatus(event, 500)
Copy link
Contributor

Choose a reason for hiding this comment

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

Why 500? This is not a server error

@Prospector
Copy link
Member

Superseded by #1225

@Prospector Prospector closed this Jan 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants