Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Please add stripping of duplicates in FeedMergeBridge #2855

Closed
HolgerAusB opened this issue Jun 22, 2022 · 2 comments · Fixed by #2888
Closed

Please add stripping of duplicates in FeedMergeBridge #2855

HolgerAusB opened this issue Jun 22, 2022 · 2 comments · Fixed by #2888
Labels
Bridge-Request Request for a new bridge

Comments

@HolgerAusB
Copy link

Bridge additional feature request

FeedMerge Bridge of @dvikan is great! But I have some problems to understand some things.

It gets 10 Items of up to 5 feeds and combine them to a single feed. But RSS-Bridge is limiting the outgoing feed to 10 items. So most likely from the 3rd to 5th feed no items might be in the feed. Or does the bridge resort all articles by publishing date and only the 10 newest will go to the combined outgoing feed?

Would it be hard to have an option to strip duplicate articles? Then this bridge would make a lot more sense.

Examples

The following two feeds have lots of identical articles because news about the capital city 'Vienna' (Wien) are also news within the country of 'Austria' (Österreich).

@HolgerAusB HolgerAusB added the Bridge-Request Request for a new bridge label Jun 22, 2022
@dvikan
Copy link
Contributor

dvikan commented Jun 22, 2022

See #2854

Will considering de-duplication. Maybe a check for it.

dvikan added a commit to dvikan/rss-bridge that referenced this issue Jul 5, 2022
dvikan added a commit that referenced this issue Jul 5, 2022
@HolgerAusB
Copy link
Author

HolgerAusB commented Jul 6, 2022

Marvelous, thank you so much @dvikan!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Bridge-Request Request for a new bridge
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants