Skip to content

[WirecutterDealsBridge] Add bridge#4359

Merged
dvikan merged 1 commit intoRSS-Bridge:masterfrom
Vynce:wirecutterdeals
Dec 12, 2024
Merged

[WirecutterDealsBridge] Add bridge#4359
dvikan merged 1 commit intoRSS-Bridge:masterfrom
Vynce:wirecutterdeals

Conversation

@Vynce
Copy link
Contributor

@Vynce Vynce commented Dec 7, 2024

Wirecutter only provides an RSS feed for their articles. This bridge creates a feed for their deals.

@github-actions
Copy link

github-actions bot commented Dec 7, 2024

Pull request artifacts

Bridge Context Status
WirecutterDeals 1 untitled (pr) ✔️

last change: Sunday 2024-12-08 22:00:01

@Vynce
Copy link
Contributor Author

Vynce commented Dec 7, 2024

The embedded icon is a teal version of their regular favicon to match their deals icon on X. This icon isn't available anywhere, so I chose to embed it. The embedded icon only shows up in the Atom feed -- I don't see it in the Mrss feed (maybe a bug?).

@Vynce
Copy link
Contributor Author

Vynce commented Dec 7, 2024

I've had problems with FreshRSS 1.24.3 generating duplicate items from this feed, particularly when it's changing frequently (Black Friday). The FreshRSS log has entries about invalid GUIDs every time this happens:

[Tue, 03 Dec 2024 09:45:06 -0600] [warning] --- Feed has valid GUIDs again: http://rss-bridge/?action=display&bridge=WirecutterDealsBridge&format=Mrss
[Tue, 03 Dec 2024 10:30:13 -0600] [warning] --- Feed has invalid GUIDs: http://rss-bridge/?action=display&bridge=WirecutterDealsBridge&format=Mrss

I haven't been able to track down this issue so far. My only thought is that rss-bridge is falling back to URL-based GUIDs from sha1-based GUIDs sometimes (?). It might also be fixed by FreshRSS/FreshRSS#4487.

@dvikan
Copy link
Contributor

dvikan commented Dec 8, 2024

move collectData method to top so class can be read like a book

@dvikan dvikan merged commit f0db6a2 into RSS-Bridge:master Dec 12, 2024
@Vynce Vynce deleted the wirecutterdeals branch December 13, 2024 03:02
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