{"action":"created","issue":{"url":"https://api.github.com/repos/alphagov/smart-answers/issues/1710","labels_url":"https://api.github.com/repos/alphagov/smart-answers/issues/1710/labels{/name}","comments_url":"https://api.github.com/repos/alphagov/smart-answers/issues/1710/comments","events_url":"https://api.github.com/repos/alphagov/smart-answers/issues/1710/events","html_url":"https://github.com/alphagov/smart-answers/pull/1710","id":88468976,"number":1710,"title":"Raise exceptions when marriage abroad is misused","user":{"login":"tadast","id":218239,"avatar_url":"https://avatars.githubusercontent.com/u/218239?v=3","gravatar_id":"","url":"https://api.github.com/users/tadast","html_url":"https://github.com/tadast","followers_url":"https://api.github.com/users/tadast/followers","following_url":"https://api.github.com/users/tadast/following{/other_user}","gists_url":"https://api.github.com/users/tadast/gists{/gist_id}","starred_url":"https://api.github.com/users/tadast/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tadast/subscriptions","organizations_url":"https://api.github.com/users/tadast/orgs","repos_url":"https://api.github.com/users/tadast/repos","events_url":"https://api.github.com/users/tadast/events{/privacy}","received_events_url":"https://api.github.com/users/tadast/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-06-15T16:02:19Z","updated_at":"2015-06-18T10:01:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/alphagov/smart-answers/pulls/1710","html_url":"https://github.com/alphagov/smart-answers/pull/1710","diff_url":"https://github.com/alphagov/smart-answers/pull/1710.diff","patch_url":"https://github.com/alphagov/smart-answers/pull/1710.patch"},"body":"I added a couple of errors to inform developers when they're doing things in a manner that is unexpected by the existing logic.\r\n\r\nDetailed explanations can be found in the commit messages.\r\n\r\nPro tip: add `?w=1` to the url param to ignore withespace changes when reviewing."},"comment":{"url":"https://api.github.com/repos/alphagov/smart-answers/issues/comments/113097424","html_url":"https://github.com/alphagov/smart-answers/pull/1710#issuecomment-113097424","issue_url":"https://api.github.com/repos/alphagov/smart-answers/issues/1710","id":113097424,"user":{"login":"chrisroos","id":6556,"avatar_url":"https://avatars.githubusercontent.com/u/6556?v=3","gravatar_id":"","url":"https://api.github.com/users/chrisroos","html_url":"https://github.com/chrisroos","followers_url":"https://api.github.com/users/chrisroos/followers","following_url":"https://api.github.com/users/chrisroos/following{/other_user}","gists_url":"https://api.github.com/users/chrisroos/gists{/gist_id}","starred_url":"https://api.github.com/users/chrisroos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chrisroos/subscriptions","organizations_url":"https://api.github.com/users/chrisroos/orgs","repos_url":"https://api.github.com/users/chrisroos/repos","events_url":"https://api.github.com/users/chrisroos/events{/privacy}","received_events_url":"https://api.github.com/users/chrisroos/received_events","type":"User","site_admin":false},"created_at":"2015-06-18T10:01:07Z","updated_at":"2015-06-18T10:01:07Z","body":"> The whole reason for the change is basically to make it obvious that this outcome does not have a default set of phrases.\r\n>\r\n> Having said it all I do not feel strongly about it, but I believe it's more useful than harmful.\r\n\r\nThat sounds entirely sensible to me. I hadn't considered the case you've demonstrated with the multiple `if ... end`'s versus the `if ... elsif ... else ... end`'s."}}