-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[TheFarSideBridge] Add Bridge #1484
Conversation
LGTM. Hard to understand that GoComics built an entire new site, instead of being a regular comic on their platform, we already have a GoComics bridge. |
Is there anything in particular keeping this from getting merged? Thinking that The Far Side was already supported was one of the main reasons that I installed RSS Bridge. |
I don't know exactly who has commit access, I'm just a user and run an instance myself. |
@em92 can you merge this, please. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, @VerifiedJoseph ! See suggestions below
@Poorchop for some reason, this PR was not in my notifications until VerifiedJoseph metioned me. |
Thanks for this, it's awesome. Thanks in general to everyone who helped develop and maintain this project. |
Adds bridge for The Far Side's Daily Dose.
To get around the hotlink protection, images are downloaded, encoded as base64 and then added to the HTML.
Closes #1403