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

Add RSS feeds (fix #57) #90

Merged
merged 7 commits into from
Jul 21, 2024
Merged

Add RSS feeds (fix #57) #90

merged 7 commits into from
Jul 21, 2024

Conversation

sigaloid
Copy link
Member

@sigaloid sigaloid commented Apr 9, 2024

Fixes #57

image

Links don't work because we don't know exactly the full URL (can't do relative linking in RSS).

Todo:

  • Config/compile gate
  • Add config option for full domain URL (needs to be placed in relevant URLs)
  • Properly add all RSS fields to better match official Reddit RSS feed, though some are flat out incompatible with (this) RSS library (rss - https://lib.rs/rss).

@nickelswitte
Copy link

Super nice you are working on this! thanks a lot!

@sigaloid sigaloid marked this pull request as ready for review July 21, 2024 18:02
@sigaloid
Copy link
Member Author

sigaloid commented Jul 21, 2024

Live from the Boston Rust hackathon...

image
Tada!
REDLIB_ENABLE_RSS=on REDLIB_FULL_URL=http://localhost:8080 cargo r (FULL_URL is necessary if you want the link to be accurate for self posts)

@sigaloid sigaloid merged commit 374238a into main Jul 21, 2024
3 checks passed
@sigaloid sigaloid deleted the rss branch July 21, 2024 18:09
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.

✨ Feature parity: RSS feeds
2 participants