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

Add additional fake news sites #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

SeanKilleen
Copy link

@SeanKilleen SeanKilleen commented Nov 20, 2016

Saw this on HN and figured I'd suggest a few more:

From this Snopes field guide to fake news:

Saw this on HN and figured I'd suggest a few more:

From this [Snopes field guide to fake news](http://www.snopes.com/2016/01/14/fake-news-sites/): 

* [National Report](http://nationalreport.net/) [[Admittance of fake news](http://nationalreport.net/disclaimer/))]
* [News Examiner](http://newsexaminer.net/) [[RealOrSatire confirmation](http://realorsatire.com/newsexaminer-net/)]
@SeanKilleen
Copy link
Author

Sorry, didn't have time to add more than a few (in the middle of something else).

But I love this idea! I'll add some other suggested sources of finding fake news so that we can import sites from them over time.

@jacquerie
Copy link
Owner

Sorry, didn't have time to add more than a few (in the middle of something else).

Thanks for this PR! I'm not going to merge it right now, because I've been awake for more than 24 hours for the hackathon, and I really need to get some sleep. I promise I will look at this tomorrow around 18:00 CET.

@@ -32,6 +32,8 @@ chrome.webRequest.onBeforeRequest.addListener(
'*://*.usconservativetoday.com/*',
'*://*.worldnewsdailyreport.com/*',
'*://*.worldpoliticus.com/*',
'*://*.nationalreport.net/*',
'*://*.newsexaminer.net/*'
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Argh, I see that you fell victim of a shortcut I took during the hackathon. There's actually two block lists, one in background.js and one in facebook.js.

They actually (should) mirror their content, so this part should be refactored to use a common list.

@jacquerie
Copy link
Owner

I was on the fence about Paul Horner, because it almost looks like he's some kind of performance artist. But yeah, he should probably go in the list.

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