Skip to content

Rearranged domains.js and added inline docs

Shosh Vasserman requested to merge domains_rearrange into master

Following the discussion on Slack, I made the following cosmetic changes to domains.js in order to make it easier for casual viewers of the code to understand what the domain lists are.

  1. Change the order of the two explicit lists of sites (so that news sites are first)
  2. Provide a comment that explains what the different domain lists represent in terms of the language of the Current Studies page
  • destinationDomains <-> news outlets
  • sensitiveDomains <-> news aggregators, search engines, and social media websites
  1. Added in-line comments matching the description of what we collect for each set of domains

Merge request reports