Skip to content

[BinanceBridge] Remove announcements because of Cloudflare issue#2610

Merged
Bockiii merged 10 commits intoRSS-Bridge:masterfrom
thefranke:BinanceBridge
Apr 5, 2022
Merged

[BinanceBridge] Remove announcements because of Cloudflare issue#2610
Bockiii merged 10 commits intoRSS-Bridge:masterfrom
thefranke:BinanceBridge

Conversation

@thefranke
Copy link
Contributor

This update removes the announcements category from the bridge, since the Zendesk source is protected by Cloudflare. After this update the bridge will only fetch the blog.

@github-actions
Copy link

github-actions bot commented Apr 5, 2022

Pull request artifacts

file last change
Binance-current-context1 2022-04-05, 16:59:21
Binance-pr-context1 2022-04-05, 16:59:21

@Bockiii
Copy link
Contributor

Bockiii commented Apr 5, 2022

Can't you also get rid of geturl and collectblogdata and put it all in collectdata? That would make the bridge easier to read and understand.

@thefranke
Copy link
Contributor Author

Can't you also get rid of geturl and collectblogdata and put it all in collectdata? That would make the bridge easier to read and understand.

Done.

@Bockiii
Copy link
Contributor

Bockiii commented Apr 5, 2022

Awesome, love it, thanks!

@Bockiii Bockiii merged commit a3b0b91 into RSS-Bridge:master Apr 5, 2022
IAM-marco pushed a commit to IAM-marco/rss-bridge that referenced this pull request Jun 17, 2022
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