search

RSS-Bridge / rss-bridge

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

[ScribbleHubBridge] Add list page feed creation #4012

Closed Phantop closed 2 months ago

Phantop commented 3 months ago

Adds a new context for generating ScribbleHub feeds from typical story lists, particularly those made by the website's series finder. Fetches contents of the first chapter of each series, as well.

github-actions[bot] commented 3 months ago

Pull request artifacts

Bridge Context Status
ScribbleHub 1 All (current) ✔️
ScribbleHub 1 All (pr) ✔️
ScribbleHub 2 Author (current) ⚠️ (shutdown) 8192: Creation of dynamic property ScribbleHubBridge::$author is deprecated in bridges/ScribbleHubBridge.php line 56
ScribbleHub 2 Author (pr) ✔️
ScribbleHub 3 Series (current) ⚠️ The feed has no items
ScribbleHub 3 Series (pr) ⚠️ The feed has no items
ScribbleHub 4 List (pr) ❌ HTTP status 504 Gateway Time-out

last change: Sunday 2024-04-07 20:11:04

dvikan commented 3 months ago

the Pull request artifacts has some errors. unclear if we should ignore them or not

Phantop commented 3 months ago

the Pull request artifacts has some errors. unclear if we should ignore them or not

Hmm that's odd. I'm running this PR on my own RSS-Bridge instance and the example link works just fine? And the other contexts are accessing content just fine so it doesn't seem to be Cloudflare (the List context requires getting past Cloudflare to work, whereas the other two just need it to fill in entry contents).

I'd likely say ignore? Since it does work when being actually used, afaict.

dvikan commented 3 months ago 
8192: Creation of dynamic property ScribbleHubBridge::$author is deprecated in bridges/ScribbleHubBridge.php line 56

this is a deprecation notice. will produce exception in later php versions. it might already be an exception in php 8.3 or 8.4. im not sure dont remember. better to fix it now if you have time