search

RSS-Bridge / rss-bridge

The RSS feed for websites missing it
https://rss-bridge.org/bridge01/
The Unlicense
7k stars 1.02k forks source link

Update RedditBridge.php #3964

Closed Tone866 closed 4 months ago

Tone866 commented 4 months ago

I read my rss mostly on mobile and old.reddit.com isn't very good to read on mobile devices. So I added an option to change the article link to the "new" reddit.

github-actions[bot] commented 4 months ago

Pull request artifacts

Bridge Context Status
Reddit 1 single (current) ⚠️ (shutdown) 8192: str_replace(): Passing null to parameter #3 ($subject) of type arraystring is deprecated in bridges/RedditBridge.php line 130
Reddit 1 single (pr) ⚠️ (shutdown) 8192: str_replace(): Passing null to parameter #3 ($subject) of type arraystring is deprecated in bridges/RedditBridge.php line 144
Reddit 2 multi (current) ⚠️ (shutdown) 8192: str_replace(): Passing null to parameter #3 ($subject) of type arraystring is deprecated in bridges/RedditBridge.php line 130
Reddit 2 multi (pr) ⚠️ (shutdown) 8192: str_replace(): Passing null to parameter #3 ($subject) of type arraystring is deprecated in bridges/RedditBridge.php line 144
Reddit 3 user (current) ⚠️ (shutdown) 8192: str_replace(): Passing null to parameter #3 ($subject) of type arraystring is deprecated in bridges/RedditBridge.php line 130
Reddit 3 user (pr) ⚠️ (shutdown) 8192: str_replace(): Passing null to parameter #3 ($subject) of type arraystring is deprecated in bridges/RedditBridge.php line 144

last change: Thursday 2024-02-08 19:02:14

dvikan commented 4 months ago

good idea.

can you change this into a list? because there are multiple frontends e.g. also https://libreddit.kavin.rocks/r/python

Tone866 commented 4 months ago

good idea.

can you change this into a list? because there are multiple frontends e.g. also https://libreddit.kavin.rocks/r/python

Ok, I changed it to a more generic list for all frontends

dvikan commented 4 months ago

this pr breaks existing feed urls because they dont have the frontend parameter.

Tone866 commented 4 months ago

existing feeds should now work as well