{"action":"created","comment":{"url":"https://api.github.com/repos/moneyadviceservice/frontend/pulls/comments/23998106","id":23998106,"diff_hunk":"@@ -0,0 +1,93 @@\n+module NewsletterExclusions\n+  def self.slugs\n+    [\n+      'making-or-reviewing-your-will-after-someone-dies',\n+      'coping-with-debt-after-your-partner-dies',\n+      'claiming-bereavement-and-other-benefits-after-someone-dies',\n+      'coping-with-a-drop-in-household-income-after-your-partner-dies',\n+      'dealing-with-the-debts-of-someone-who-has-died',\n+      'when-should-you-use-a-probate-specialist',\n+      'calculating-and-paying-tax-after-someone-dies',\n+      'what-to-do-about-someones-pension-when-theyve-died',\n+      'how-to-work-out-the-value-of-someones-estate',\n+      'paying-for-a-funeral',\n+      'funeral-plans',\n+      'how-much-does-a-funeral-cost',\n+      'pension-and-retirement-planning-following-a-bereavement',\n+      'coping-with-grief-when-someone-dies',\n+      'reviewing-your-insurance-after-a-bereavement',\n+      'what-to-do-when-someone-dies-a-checklist',\n+      'reviewing-your-savings-and-investments-after-a-bereavement',\n+      'making-the-most-of-your-inheritance',\n+      'budgeting-after-your-partner-dies',\n+      'taking-over-the-budget-after-your-partner-dies',\n+      'sorting-out-someones-estate-where-to-start',\n+      'help-manage-the-money-of-someone-youre-caring-for',\n+      'support-services-available-to-carers',\n+      'financial-support-for-young-carers',\n+      'benefits-and-tax-credits-you-can-claim-as-a-carer',\n+      'when-does-someone-need-formal-help-with-managing-their-money',\n+      'help-someone-informally-with-budgeting-and-day-to-day-money',\n+      'if-the-person-you-want-to-help-can-still-manage-their-money',\n+      'if-the-person-you-want-to-help-has-lost-mental-capacity',\n+      'how-to-sort-out-your-money-if-you-become-ill-or-disabled',\n+      'support-to-help-you-keep-your-job-when-ill-or-disabled',\n+      'help-to-find-work-if-youre-disabled',\n+      'support-to-help-you-study-when-ill-or-disabled',\n+      'disability-and-sickness-benefits-check-your-entitlements',\n+      'personal-independence-payment-an-introduction',\n+      'universal-credit-for-disabled-people',\n+      'financial-support-if-you-or-your-child-has-a-disability',\n+      'how-to-appeal-against-a-disability-benefits-decision',\n+      'funding-to-adapt-your-home-for-accessibility',\n+      'charitable-grants-for-ill-or-disabled-people',\n+      'shopping-around-for-disability-aids-and-equipment',\n+      'motability-blue-badge-scheme-and-discounted-travel',\n+      'money-saving-tips-and-discounts-for-disabled-people',\n+      'buying-insurance-if-youre-ill-or-disabled',\n+      'getting-a-mortgage-if-youre-ill-or-disabled',\n+      'getting-a-loan-if-you-are-ill-or-disabled',\n+      'early-retirement-because-of-illness-or-disability',\n+      'long-term-care-services-at-a-glance',\n+      'planning-ahead-for-when-you-cant-manage-your-money',\n+      'care-services-to-help-you-stay-in-your-own-home',\n+      'preparing-for-illness-old-age-and-death',\n+      'care-home-or-home-care',\n+      'choosing-the-right-care-home',\n+      'what-if-youre-unhappy-with-the-care-you-receive',\n+      'how-to-challenge-your-local-authority-over-your-care',\n+      'solving-problems-when-family-members-are-helping-with-your-money',\n+      'using-a-personal-assistant-to-provide-your-care',\n+      'when-does-someone-need-formal-help-with-managing-their-money',\n+      'setting-up-a-power-of-attorney',\n+      'setting-up-a-trust',\n+      'being-a-trustee',\n+      'resolving-problems-with-attorneys-and-trustees',\n+      'self-funding-your-long-term-care-your-options',\n+      'are-you-eligible-for-nhs-continuing-care-funding',\n+      'benefits-you-can-claim-when-you-have-care-needs',\n+      'get-financial-advice-on-how-to-fund-your-long-term-care',\n+      'local-authority-funding-for-care-costs-do-you-qualify',\n+      'registered-nursing-care-contribution',\n+      'means-tests-for-help-with-care-costs-how-they-work',\n+      'how-a-local-authority-care-needs-assessment-works',\n+      'direct-payments-explained',\n+      'managing-direct-payments-to-pay-for-the-care-you-need',\n+      'how-to-fund-your-long-term-care-a-beginners-guide',\n+      'claiming-on-insurance-to-help-cover-the-costs-of-care',\n+      'using-a-home-reversion-plan-to-pay-for-your-care',\n+      'make-your-money-easier-to-manage-by-yourself',\n+      'using-investment-bonds-to-pay-for-your-long-term-care',\n+      'immediate-need-care-fee-payment-plans',\n+      'using-a-lifetime-mortgage-to-pay-for-your-long-term-care',\n+      'using-an-equity-release-scheme-to-fund-your-care',\n+      'paying-your-own-care-costs-but-the-moneys-run-out',\n+      'downsizing-your-home-to-fund-your-long-term-care',\n+      'if-your-baby-is-stillborn',\n+      'if-the-person-you-want-to-help-has-lost-mental-capacity',\n+      'putting-your-affairs-in-order',\n+      'preparing-for-illness-old-age-and-death',\n+      'choosing-your-executor'\n+    ]\n+  end","path":"lib/newsletter_exclusion.rb","position":92,"original_position":92,"commit_id":"d0905640a7dc4112e24138d03276b9f9ff35b579","original_commit_id":"d0905640a7dc4112e24138d03276b9f9ff35b579","user":{"login":"tomas-stefano","id":27509,"avatar_url":"https://avatars.githubusercontent.com/u/27509?v=3","gravatar_id":"","url":"https://api.github.com/users/tomas-stefano","html_url":"https://github.com/tomas-stefano","followers_url":"https://api.github.com/users/tomas-stefano/followers","following_url":"https://api.github.com/users/tomas-stefano/following{/other_user}","gists_url":"https://api.github.com/users/tomas-stefano/gists{/gist_id}","starred_url":"https://api.github.com/users/tomas-stefano/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomas-stefano/subscriptions","organizations_url":"https://api.github.com/users/tomas-stefano/orgs","repos_url":"https://api.github.com/users/tomas-stefano/repos","events_url":"https://api.github.com/users/tomas-stefano/events{/privacy}","received_events_url":"https://api.github.com/users/tomas-stefano/received_events","type":"User","site_admin":false},"body":"I would move this to some data store instead hard-coded data on a module that exclude slugs from the newsletter. ","created_at":"2015-02-03T11:40:30Z","updated_at":"2015-02-03T11:40:30Z","html_url":"https://github.com/moneyadviceservice/frontend/pull/889#discussion_r23998106","pull_request_url":"https://api.github.com/repos/moneyadviceservice/frontend/pulls/889","_links":{"self":{"href":"https://api.github.com/repos/moneyadviceservice/frontend/pulls/comments/23998106"},"html":{"href":"https://github.com/moneyadviceservice/frontend/pull/889#discussion_r23998106"},"pull_request":{"href":"https://api.github.com/repos/moneyadviceservice/frontend/pulls/889"}}},"pull_request":{"url":"https://api.github.com/repos/moneyadviceservice/frontend/pulls/889","id":28480740,"html_url":"https://github.com/moneyadviceservice/frontend/pull/889","diff_url":"https://github.com/moneyadviceservice/frontend/pull/889.diff","patch_url":"https://github.com/moneyadviceservice/frontend/pull/889.patch","issue_url":"https://api.github.com/repos/moneyadviceservice/frontend/issues/889","number":889,"state":"open","locked":false,"title":"adding Newsletter injection script","user":{"login":"Guntrisoft","id":6049076,"avatar_url":"https://avatars.githubusercontent.com/u/6049076?v=3","gravatar_id":"","url":"https://api.github.com/users/Guntrisoft","html_url":"https://github.com/Guntrisoft","followers_url":"https://api.github.com/users/Guntrisoft/followers","following_url":"https://api.github.com/users/Guntrisoft/following{/other_user}","gists_url":"https://api.github.com/users/Guntrisoft/gists{/gist_id}","starred_url":"https://api.github.com/users/Guntrisoft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Guntrisoft/subscriptions","organizations_url":"https://api.github.com/users/Guntrisoft/orgs","repos_url":"https://api.github.com/users/Guntrisoft/repos","events_url":"https://api.github.com/users/Guntrisoft/events{/privacy}","received_events_url":"https://api.github.com/users/Guntrisoft/received_events","type":"User","site_admin":false},"body":"- uses dough loader to kick off javascript component\r\n- component injects newsletter div into the right\r\n  place within an article and adds a class\r\n  which displays it to the user\r\n- altered where the newsletter div gets included\r\n  conditionally loading it in an article\r\n  template as long as the current URL\r\n  doesn't match an excluded list of slugs\r\n  or is a welsh article","created_at":"2015-02-02T16:22:10Z","updated_at":"2015-02-03T11:40:30Z","closed_at":null,"merged_at":null,"merge_commit_sha":"a687286343de8f476a4f85ea68b49fff7489c3dc","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/moneyadviceservice/frontend/pulls/889/commits","review_comments_url":"https://api.github.com/repos/moneyadviceservice/frontend/pulls/889/comments","review_comment_url":"https://api.github.com/repos/moneyadviceservice/frontend/pulls/comments/{number}","comments_url":"https://api.github.com/repos/moneyadviceservice/frontend/issues/889/comments","statuses_url":"https://api.github.com/repos/moneyadviceservice/frontend/statuses/d0905640a7dc4112e24138d03276b9f9ff35b579","head":{"label":"moneyadviceservice:newsletter_signup_in_article","ref":"newsletter_signup_in_article","sha":"d0905640a7dc4112e24138d03276b9f9ff35b579","user":{"login":"moneyadviceservice","id":2064781,"avatar_url":"https://avatars.githubusercontent.com/u/2064781?v=3","gravatar_id":"","url":"https://api.github.com/users/moneyadviceservice","html_url":"https://github.com/moneyadviceservice","followers_url":"https://api.github.com/users/moneyadviceservice/followers","following_url":"https://api.github.com/users/moneyadviceservice/following{/other_user}","gists_url":"https://api.github.com/users/moneyadviceservice/gists{/gist_id}","starred_url":"https://api.github.com/users/moneyadviceservice/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/moneyadviceservice/subscriptions","organizations_url":"https://api.github.com/users/moneyadviceservice/orgs","repos_url":"https://api.github.com/users/moneyadviceservice/repos","events_url":"https://api.github.com/users/moneyadviceservice/events{/privacy}","received_events_url":"https://api.github.com/users/moneyadviceservice/received_events","type":"Organization","site_admin":false},"repo":{"id":15315123,"name":"frontend","full_name":"moneyadviceservice/frontend","owner":{"login":"moneyadviceservice","id":2064781,"avatar_url":"https://avatars.githubusercontent.com/u/2064781?v=3","gravatar_id":"","url":"https://api.github.com/users/moneyadviceservice","html_url":"https://github.com/moneyadviceservice","followers_url":"https://api.github.com/users/moneyadviceservice/followers","following_url":"https://api.github.com/users/moneyadviceservice/following{/other_user}","gists_url":"https://api.github.com/users/moneyadviceservice/gists{/gist_id}","starred_url":"https://api.github.com/users/moneyadviceservice/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/moneyadviceservice/subscriptions","organizations_url":"https://api.github.com/users/moneyadviceservice/orgs","repos_url":"https://api.github.com/users/moneyadviceservice/repos","events_url":"https://api.github.com/users/moneyadviceservice/events{/privacy}","received_events_url":"https://api.github.com/users/moneyadviceservice/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/moneyadviceservice/frontend","description":"The Money Advice Service's improved website.","fork":false,"url":"https://api.github.com/repos/moneyadviceservice/frontend","forks_url":"https://api.github.com/repos/moneyadviceservice/frontend/forks","keys_url":"https://api.github.com/repos/moneyadviceservice/frontend/keys{/key_id}","collaborators_url":"https://api.github.com/repos/moneyadviceservice/frontend/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/moneyadviceservice/frontend/teams","hooks_url":"https://api.github.com/repos/moneyadviceservice/frontend/hooks","issue_events_url":"https://api.github.com/repos/moneyadviceservice/frontend/issues/events{/number}","events_url":"https://api.github.com/repos/moneyadviceservice/frontend/events","assignees_url":"https://api.github.com/repos/moneyadviceservice/frontend/assignees{/user}","branches_url":"https://api.github.com/repos/moneyadviceservice/frontend/branches{/branch}","tags_url":"https://api.github.com/repos/moneyadviceservice/frontend/tags","blobs_url":"https://api.github.com/repos/moneyadviceservice/frontend/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/moneyadviceservice/frontend/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/moneyadviceservice/frontend/git/refs{/sha}","trees_url":"https://api.github.com/repos/moneyadviceservice/frontend/git/trees{/sha}","statuses_url":"https://api.github.com/repos/moneyadviceservice/frontend/statuses/{sha}","languages_url":"https://api.github.com/repos/moneyadviceservice/frontend/languages","stargazers_url":"https://api.github.com/repos/moneyadviceservice/frontend/stargazers","contributors_url":"https://api.github.com/repos/moneyadviceservice/frontend/contributors","subscribers_url":"https://api.github.com/repos/moneyadviceservice/frontend/subscribers","subscription_url":"https://api.github.com/repos/moneyadviceservice/frontend/subscription","commits_url":"https://api.github.com/repos/moneyadviceservice/frontend/commits{/sha}","git_commits_url":"https://api.github.com/repos/moneyadviceservice/frontend/git/commits{/sha}","comments_url":"https://api.github.com/repos/moneyadviceservice/frontend/comments{/number}","issue_comment_url":"https://api.github.com/repos/moneyadviceservice/frontend/issues/comments/{number}","contents_url":"https://api.github.com/repos/moneyadviceservice/frontend/contents/{+path}","compare_url":"https://api.github.com/repos/moneyadviceservice/frontend/compare/{base}...{head}","merges_url":"https://api.github.com/repos/moneyadviceservice/frontend/merges","archive_url":"https://api.github.com/repos/moneyadviceservice/frontend/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/moneyadviceservice/frontend/downloads","issues_url":"https://api.github.com/repos/moneyadviceservice/frontend/issues{/number}","pulls_url":"https://api.github.com/repos/moneyadviceservice/frontend/pulls{/number}","milestones_url":"https://api.github.com/repos/moneyadviceservice/frontend/milestones{/number}","notifications_url":"https://api.github.com/repos/moneyadviceservice/frontend/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/moneyadviceservice/frontend/labels{/name}","releases_url":"https://api.github.com/repos/moneyadviceservice/frontend/releases{/id}","created_at":"2013-12-19T15:25:21Z","updated_at":"2015-02-03T11:32:56Z","pushed_at":"2015-02-03T11:32:55Z","git_url":"git://github.com/moneyadviceservice/frontend.git","ssh_url":"git@github.com:moneyadviceservice/frontend.git","clone_url":"https://github.com/moneyadviceservice/frontend.git","svn_url":"https://github.com/moneyadviceservice/frontend","homepage":"https://www.moneyadviceservice.org.uk  ","size":29889,"stargazers_count":3,"watchers_count":3,"language":"Ruby","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"open_issues_count":10,"forks":3,"open_issues":10,"watchers":3,"default_branch":"master"}},"base":{"label":"moneyadviceservice:master","ref":"master","sha":"dff42868ca7e48fab3889cbfb7127f9987b0b507","user":{"login":"moneyadviceservice","id":2064781,"avatar_url":"https://avatars.githubusercontent.com/u/2064781?v=3","gravatar_id":"","url":"https://api.github.com/users/moneyadviceservice","html_url":"https://github.com/moneyadviceservice","followers_url":"https://api.github.com/users/moneyadviceservice/followers","following_url":"https://api.github.com/users/moneyadviceservice/following{/other_user}","gists_url":"https://api.github.com/users/moneyadviceservice/gists{/gist_id}","starred_url":"https://api.github.com/users/moneyadviceservice/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/moneyadviceservice/subscriptions","organizations_url":"https://api.github.com/users/moneyadviceservice/orgs","repos_url":"https://api.github.com/users/moneyadviceservice/repos","events_url":"https://api.github.com/users/moneyadviceservice/events{/privacy}","received_events_url":"https://api.github.com/users/moneyadviceservice/received_events","type":"Organization","site_admin":false},"repo":{"id":15315123,"name":"frontend","full_name":"moneyadviceservice/frontend","owner":{"login":"moneyadviceservice","id":2064781,"avatar_url":"https://avatars.githubusercontent.com/u/2064781?v=3","gravatar_id":"","url":"https://api.github.com/users/moneyadviceservice","html_url":"https://github.com/moneyadviceservice","followers_url":"https://api.github.com/users/moneyadviceservice/followers","following_url":"https://api.github.com/users/moneyadviceservice/following{/other_user}","gists_url":"https://api.github.com/users/moneyadviceservice/gists{/gist_id}","starred_url":"https://api.github.com/users/moneyadviceservice/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/moneyadviceservice/subscriptions","organizations_url":"https://api.github.com/users/moneyadviceservice/orgs","repos_url":"https://api.github.com/users/moneyadviceservice/repos","events_url":"https://api.github.com/users/moneyadviceservice/events{/privacy}","received_events_url":"https://api.github.com/users/moneyadviceservice/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/moneyadviceservice/frontend","description":"The Money Advice Service's improved website.","fork":false,"url":"https://api.github.com/repos/moneyadviceservice/frontend","forks_url":"https://api.github.com/repos/moneyadviceservice/frontend/forks","keys_url":"https://api.github.com/repos/moneyadviceservice/frontend/keys{/key_id}","collaborators_url":"https://api.github.com/repos/moneyadviceservice/frontend/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/moneyadviceservice/frontend/teams","hooks_url":"https://api.github.com/repos/moneyadviceservice/frontend/hooks","issue_events_url":"https://api.github.com/repos/moneyadviceservice/frontend/issues/events{/number}","events_url":"https://api.github.com/repos/moneyadviceservice/frontend/events","assignees_url":"https://api.github.com/repos/moneyadviceservice/frontend/assignees{/user}","branches_url":"https://api.github.com/repos/moneyadviceservice/frontend/branches{/branch}","tags_url":"https://api.github.com/repos/moneyadviceservice/frontend/tags","blobs_url":"https://api.github.com/repos/moneyadviceservice/frontend/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/moneyadviceservice/frontend/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/moneyadviceservice/frontend/git/refs{/sha}","trees_url":"https://api.github.com/repos/moneyadviceservice/frontend/git/trees{/sha}","statuses_url":"https://api.github.com/repos/moneyadviceservice/frontend/statuses/{sha}","languages_url":"https://api.github.com/repos/moneyadviceservice/frontend/languages","stargazers_url":"https://api.github.com/repos/moneyadviceservice/frontend/stargazers","contributors_url":"https://api.github.com/repos/moneyadviceservice/frontend/contributors","subscribers_url":"https://api.github.com/repos/moneyadviceservice/frontend/subscribers","subscription_url":"https://api.github.com/repos/moneyadviceservice/frontend/subscription","commits_url":"https://api.github.com/repos/moneyadviceservice/frontend/commits{/sha}","git_commits_url":"https://api.github.com/repos/moneyadviceservice/frontend/git/commits{/sha}","comments_url":"https://api.github.com/repos/moneyadviceservice/frontend/comments{/number}","issue_comment_url":"https://api.github.com/repos/moneyadviceservice/frontend/issues/comments/{number}","contents_url":"https://api.github.com/repos/moneyadviceservice/frontend/contents/{+path}","compare_url":"https://api.github.com/repos/moneyadviceservice/frontend/compare/{base}...{head}","merges_url":"https://api.github.com/repos/moneyadviceservice/frontend/merges","archive_url":"https://api.github.com/repos/moneyadviceservice/frontend/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/moneyadviceservice/frontend/downloads","issues_url":"https://api.github.com/repos/moneyadviceservice/frontend/issues{/number}","pulls_url":"https://api.github.com/repos/moneyadviceservice/frontend/pulls{/number}","milestones_url":"https://api.github.com/repos/moneyadviceservice/frontend/milestones{/number}","notifications_url":"https://api.github.com/repos/moneyadviceservice/frontend/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/moneyadviceservice/frontend/labels{/name}","releases_url":"https://api.github.com/repos/moneyadviceservice/frontend/releases{/id}","created_at":"2013-12-19T15:25:21Z","updated_at":"2015-02-03T11:32:56Z","pushed_at":"2015-02-03T11:32:55Z","git_url":"git://github.com/moneyadviceservice/frontend.git","ssh_url":"git@github.com:moneyadviceservice/frontend.git","clone_url":"https://github.com/moneyadviceservice/frontend.git","svn_url":"https://github.com/moneyadviceservice/frontend","homepage":"https://www.moneyadviceservice.org.uk  ","size":29889,"stargazers_count":3,"watchers_count":3,"language":"Ruby","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"open_issues_count":10,"forks":3,"open_issues":10,"watchers":3,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/moneyadviceservice/frontend/pulls/889"},"html":{"href":"https://github.com/moneyadviceservice/frontend/pull/889"},"issue":{"href":"https://api.github.com/repos/moneyadviceservice/frontend/issues/889"},"comments":{"href":"https://api.github.com/repos/moneyadviceservice/frontend/issues/889/comments"},"review_comments":{"href":"https://api.github.com/repos/moneyadviceservice/frontend/pulls/889/comments"},"review_comment":{"href":"https://api.github.com/repos/moneyadviceservice/frontend/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/moneyadviceservice/frontend/pulls/889/commits"},"statuses":{"href":"https://api.github.com/repos/moneyadviceservice/frontend/statuses/d0905640a7dc4112e24138d03276b9f9ff35b579"}}}}