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

[KilledbyGoogleBridge] Add bridge #1373

Merged
merged 1 commit into from
Mar 24, 2022
Merged

[KilledbyGoogleBridge] Add bridge #1373

merged 1 commit into from
Mar 24, 2022

Conversation

VerifiedJoseph
Copy link
Contributor

Adds a bridge for Killed by Google. Uses the JSON file that powers the website. A RSS feed has requested for the website, but not yet added.

@em92
Copy link
Contributor

em92 commented Nov 27, 2019

How about implementing it in KilledByGoogle sources instead? Author does not seem to be against this.

@logmanoriginal
Copy link
Contributor

As @em92 pointed out, support for feeds is on the roadmap for KilledByGoogle. I also think it's best to have it generated by them. They also accept PRs, so no holding back 😁

@dvikan
Copy link
Contributor

dvikan commented Mar 22, 2022

Don't think they have added a feed yet. Will probably merge this.

@yamanq
Copy link
Contributor

yamanq commented Mar 22, 2022

Yes, they also closed the issue and removed the roadmap, so it's unlikely to be added in the near future.

@dvikan dvikan merged commit 296ff9c into RSS-Bridge:master Mar 24, 2022
@dvikan
Copy link
Contributor

dvikan commented Mar 24, 2022

I was confused at first why rssbridge didnt show Youtube Originals which was listed on the frontpage of killedbygoogle.com
But I know understand that the rss feed item is only generated past the actual closing date.

@Bockiii Bockiii mentioned this pull request Mar 26, 2022
Kwbmm pushed a commit to Kwbmm/rss-bridge that referenced this pull request Jun 17, 2022
@VerifiedJoseph VerifiedJoseph deleted the KilledbyGoogleBridge branch July 5, 2022 16:45
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.

5 participants