GitHub PullRequestReviewCommentEvent at 2014-09-03 19:45:24
Repository: zamboni
Actor: mstriemer
Payload: {"action":"created","comment":{"url":"https://api.github.com/repos/mozilla/zamboni/pulls/comments/17074737","id":17074737,"diff_hunk":"@@ -364,11 +367,23 @@ def flag(cls, addon, event, message=None):\n                          note_type=comm.ACTION_MAP(event))\n \n \n+def send_tarako_mail(review):\n+    if not waffle.switch_is_active('comm-dashboard'):\n+        send_mail_jinja(\n+            'Tarako review {passed}'.format(\n+                passed='passed' if review.passed else 'failed'),","path":"mkt/reviewers/models.py","position":28,"original_position":28,"commit_id":"bcefcaec4d1d29405a2b074da0b110c296d04a83","original_commit_id":"1cb496fc6ef47dc1035ea9242d2bff1a30c05717","user":{"login":"mstriemer","id":211578,"avatar_url":"https://avatars.githubusercontent.com/u/211578?v=2","gravatar_id":"2eec708f78fb85d1ec7380ba8d0f1bcb","url":"https://api.github.com/users/mstriemer","html_url":"https://github.com/mstriemer","followers_url":"https://api.github.com/users/mstriemer/followers","following_url":"https://api.github.com/users/mstriemer/following{/other_user}","gists_url":"https://api.github.com/users/mstriemer/gists{/gist_id}","starred_url":"https://api.github.com/users/mstriemer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mstriemer/subscriptions","organizations_url":"https://api.github.com/users/mstriemer/orgs","repos_url":"https://api.github.com/users/mstriemer/repos","events_url":"https://api.github.com/users/mstriemer/events{/privacy}","received_events_url":"https://api.github.com/users/mstriemer/received_events","type":"User","site_admin":false},"body":"I don't think we localize emails. If we do then it does. I checked a few other reviewer triggered emails and they don't appear to be localized so I left it out.","created_at":"2014-09-03T19:45:24Z","updated_at":"2014-09-03T19:45:24Z","html_url":"https://github.com/mozilla/zamboni/pull/2496#discussion_r17074737","pull_request_url":"https://api.github.com/repos/mozilla/zamboni/pulls/2496","_links":{"self":{"href":"https://api.github.com/repos/mozilla/zamboni/pulls/comments/17074737"},"html":{"href":"https://github.com/mozilla/zamboni/pull/2496#discussion_r17074737"},"pull_request":{"href":"https://api.github.com/repos/mozilla/zamboni/pulls/2496"}}},"pull_request":{"url":"https://api.github.com/repos/mozilla/zamboni/pulls/2496","id":20690436,"html_url":"https://github.com/mozilla/zamboni/pull/2496","diff_url":"https://github.com/mozilla/zamboni/pull/2496.diff","patch_url":"https://github.com/mozilla/zamboni/pull/2496.patch","issue_url":"https://api.github.com/repos/mozilla/zamboni/issues/2496","number":2496,"state":"open","locked":false,"title":"Email devs about Tarako reviews (bug 1053355)","user":{"login":"mstriemer","id":211578,"avatar_url":"https://avatars.githubusercontent.com/u/211578?v=2","gravatar_id":"2eec708f78fb85d1ec7380ba8d0f1bcb","url":"https://api.github.com/users/mstriemer","html_url":"https://github.com/mstriemer","followers_url":"https://api.github.com/users/mstriemer/followers","following_url":"https://api.github.com/users/mstriemer/following{/other_user}","gists_url":"https://api.github.com/users/mstriemer/gists{/gist_id}","starred_url":"https://api.github.com/users/mstriemer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mstriemer/subscriptions","organizations_url":"https://api.github.com/users/mstriemer/orgs","repos_url":"https://api.github.com/users/mstriemer/repos","events_url":"https://api.github.com/users/mstriemer/events{/privacy}","received_events_url":"https://api.github.com/users/mstriemer/received_events","type":"User","site_admin":false},"body":"https://bugzilla.mozilla.org/show_bug.cgi?id=1053355\r\n\r\n@robhudson r?","created_at":"2014-09-03T16:54:37Z","updated_at":"2014-09-03T19:45:24Z","closed_at":null,"merged_at":null,"merge_commit_sha":"8e3aea313b19c1595b00b0417ec1b769cccd2571","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/mozilla/zamboni/pulls/2496/commits","review_comments_url":"https://api.github.com/repos/mozilla/zamboni/pulls/2496/comments","review_comment_url":"https://api.github.com/repos/mozilla/zamboni/pulls/comments/{number}","comments_url":"https://api.github.com/repos/mozilla/zamboni/issues/2496/comments","statuses_url":"https://api.github.com/repos/mozilla/zamboni/statuses/bcefcaec4d1d29405a2b074da0b110c296d04a83","head":{"label":"mstriemer:email-devs-after-tarako-review-1053355","ref":"email-devs-after-tarako-review-1053355","sha":"bcefcaec4d1d29405a2b074da0b110c296d04a83","user":{"login":"mstriemer","id":211578,"avatar_url":"https://avatars.githubusercontent.com/u/211578?v=2","gravatar_id":"2eec708f78fb85d1ec7380ba8d0f1bcb","url":"https://api.github.com/users/mstriemer","html_url":"https://github.com/mstriemer","followers_url":"https://api.github.com/users/mstriemer/followers","following_url":"https://api.github.com/users/mstriemer/following{/other_user}","gists_url":"https://api.github.com/users/mstriemer/gists{/gist_id}","starred_url":"https://api.github.com/users/mstriemer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mstriemer/subscriptions","organizations_url":"https://api.github.com/users/mstriemer/orgs","repos_url":"https://api.github.com/users/mstriemer/repos","events_url":"https://api.github.com/users/mstriemer/events{/privacy}","received_events_url":"https://api.github.com/users/mstriemer/received_events","type":"User","site_admin":false},"repo":{"id":15063915,"name":"zamboni","full_name":"mstriemer/zamboni","owner":{"login":"mstriemer","id":211578,"avatar_url":"https://avatars.githubusercontent.com/u/211578?v=2","gravatar_id":"2eec708f78fb85d1ec7380ba8d0f1bcb","url":"https://api.github.com/users/mstriemer","html_url":"https://github.com/mstriemer","followers_url":"https://api.github.com/users/mstriemer/followers","following_url":"https://api.github.com/users/mstriemer/following{/other_user}","gists_url":"https://api.github.com/users/mstriemer/gists{/gist_id}","starred_url":"https://api.github.com/users/mstriemer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mstriemer/subscriptions","organizations_url":"https://api.github.com/users/mstriemer/orgs","repos_url":"https://api.github.com/users/mstriemer/repos","events_url":"https://api.github.com/users/mstriemer/events{/privacy}","received_events_url":"https://api.github.com/users/mstriemer/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/mstriemer/zamboni","description":"The Firefox Marketplace and addons.mozilla.org","fork":true,"url":"https://api.github.com/repos/mstriemer/zamboni","forks_url":"https://api.github.com/repos/mstriemer/zamboni/forks","keys_url":"https://api.github.com/repos/mstriemer/zamboni/keys{/key_id}","collaborators_url":"https://api.github.com/repos/mstriemer/zamboni/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/mstriemer/zamboni/teams","hooks_url":"https://api.github.com/repos/mstriemer/zamboni/hooks","issue_events_url":"https://api.github.com/repos/mstriemer/zamboni/issues/events{/number}","events_url":"https://api.github.com/repos/mstriemer/zamboni/events","assignees_url":"https://api.github.com/repos/mstriemer/zamboni/assignees{/user}","branches_url":"https://api.github.com/repos/mstriemer/zamboni/branches{/branch}","tags_url":"https://api.github.com/repos/mstriemer/zamboni/tags","blobs_url":"https://api.github.com/repos/mstriemer/zamboni/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/mstriemer/zamboni/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/mstriemer/zamboni/git/refs{/sha}","trees_url":"https://api.github.com/repos/mstriemer/zamboni/git/trees{/sha}","statuses_url":"https://api.github.com/repos/mstriemer/zamboni/statuses/{sha}","languages_url":"https://api.github.com/repos/mstriemer/zamboni/languages","stargazers_url":"https://api.github.com/repos/mstriemer/zamboni/stargazers","contributors_url":"https://api.github.com/repos/mstriemer/zamboni/contributors","subscribers_url":"https://api.github.com/repos/mstriemer/zamboni/subscribers","subscription_url":"https://api.github.com/repos/mstriemer/zamboni/subscription","commits_url":"https://api.github.com/repos/mstriemer/zamboni/commits{/sha}","git_commits_url":"https://api.github.com/repos/mstriemer/zamboni/git/commits{/sha}","comments_url":"https://api.github.com/repos/mstriemer/zamboni/comments{/number}","issue_comment_url":"https://api.github.com/repos/mstriemer/zamboni/issues/comments/{number}","contents_url":"https://api.github.com/repos/mstriemer/zamboni/contents/{+path}","compare_url":"https://api.github.com/repos/mstriemer/zamboni/compare/{base}...{head}","merges_url":"https://api.github.com/repos/mstriemer/zamboni/merges","archive_url":"https://api.github.com/repos/mstriemer/zamboni/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/mstriemer/zamboni/downloads","issues_url":"https://api.github.com/repos/mstriemer/zamboni/issues{/number}","pulls_url":"https://api.github.com/repos/mstriemer/zamboni/pulls{/number}","milestones_url":"https://api.github.com/repos/mstriemer/zamboni/milestones{/number}","notifications_url":"https://api.github.com/repos/mstriemer/zamboni/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/mstriemer/zamboni/labels{/name}","releases_url":"https://api.github.com/repos/mstriemer/zamboni/releases{/id}","created_at":"2013-12-10T01:24:18Z","updated_at":"2014-08-20T21:03:49Z","pushed_at":"2014-09-03T19:29:05Z","git_url":"git://github.com/mstriemer/zamboni.git","ssh_url":"git@github.com:mstriemer/zamboni.git","clone_url":"https://github.com/mstriemer/zamboni.git","svn_url":"https://github.com/mstriemer/zamboni","homepage":"http://zamboni.readthedocs.org/en/latest/","size":281611,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":false,"has_wiki":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"mozilla:master","ref":"master","sha":"3046d13fcdc5e67d9e5c4e42b73ea117819675c2","user":{"login":"mozilla","id":131524,"avatar_url":"https://avatars.githubusercontent.com/u/131524?v=2","gravatar_id":"9ba739c8f3288e256b13553e4d257b5e","url":"https://api.github.com/users/mozilla","html_url":"https://github.com/mozilla","followers_url":"https://api.github.com/users/mozilla/followers","following_url":"https://api.github.com/users/mozilla/following{/other_user}","gists_url":"https://api.github.com/users/mozilla/gists{/gist_id}","starred_url":"https://api.github.com/users/mozilla/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mozilla/subscriptions","organizations_url":"https://api.github.com/users/mozilla/orgs","repos_url":"https://api.github.com/users/mozilla/repos","events_url":"https://api.github.com/users/mozilla/events{/privacy}","received_events_url":"https://api.github.com/users/mozilla/received_events","type":"Organization","site_admin":false},"repo":{"id":347655,"name":"zamboni","full_name":"mozilla/zamboni","owner":{"login":"mozilla","id":131524,"avatar_url":"https://avatars.githubusercontent.com/u/131524?v=2","gravatar_id":"9ba739c8f3288e256b13553e4d257b5e","url":"https://api.github.com/users/mozilla","html_url":"https://github.com/mozilla","followers_url":"https://api.github.com/users/mozilla/followers","following_url":"https://api.github.com/users/mozilla/following{/other_user}","gists_url":"https://api.github.com/users/mozilla/gists{/gist_id}","starred_url":"https://api.github.com/users/mozilla/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mozilla/subscriptions","organizations_url":"https://api.github.com/users/mozilla/orgs","repos_url":"https://api.github.com/users/mozilla/repos","events_url":"https://api.github.com/users/mozilla/events{/privacy}","received_events_url":"https://api.github.com/users/mozilla/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/mozilla/zamboni","description":"The backend of the Firefox Marketplace","fork":false,"url":"https://api.github.com/repos/mozilla/zamboni","forks_url":"https://api.github.com/repos/mozilla/zamboni/forks","keys_url":"https://api.github.com/repos/mozilla/zamboni/keys{/key_id}","collaborators_url":"https://api.github.com/repos/mozilla/zamboni/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/mozilla/zamboni/teams","hooks_url":"https://api.github.com/repos/mozilla/zamboni/hooks","issue_events_url":"https://api.github.com/repos/mozilla/zamboni/issues/events{/number}","events_url":"https://api.github.com/repos/mozilla/zamboni/events","assignees_url":"https://api.github.com/repos/mozilla/zamboni/assignees{/user}","branches_url":"https://api.github.com/repos/mozilla/zamboni/branches{/branch}","tags_url":"https://api.github.com/repos/mozilla/zamboni/tags","blobs_url":"https://api.github.com/repos/mozilla/zamboni/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/mozilla/zamboni/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/mozilla/zamboni/git/refs{/sha}","trees_url":"https://api.github.com/repos/mozilla/zamboni/git/trees{/sha}","statuses_url":"https://api.github.com/repos/mozilla/zamboni/statuses/{sha}","languages_url":"https://api.github.com/repos/mozilla/zamboni/languages","stargazers_url":"https://api.github.com/repos/mozilla/zamboni/stargazers","contributors_url":"https://api.github.com/repos/mozilla/zamboni/contributors","subscribers_url":"https://api.github.com/repos/mozilla/zamboni/subscribers","subscription_url":"https://api.github.com/repos/mozilla/zamboni/subscription","commits_url":"https://api.github.com/repos/mozilla/zamboni/commits{/sha}","git_commits_url":"https://api.github.com/repos/mozilla/zamboni/git/commits{/sha}","comments_url":"https://api.github.com/repos/mozilla/zamboni/comments{/number}","issue_comment_url":"https://api.github.com/repos/mozilla/zamboni/issues/comments/{number}","contents_url":"https://api.github.com/repos/mozilla/zamboni/contents/{+path}","compare_url":"https://api.github.com/repos/mozilla/zamboni/compare/{base}...{head}","merges_url":"https://api.github.com/repos/mozilla/zamboni/merges","archive_url":"https://api.github.com/repos/mozilla/zamboni/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/mozilla/zamboni/downloads","issues_url":"https://api.github.com/repos/mozilla/zamboni/issues{/number}","pulls_url":"https://api.github.com/repos/mozilla/zamboni/pulls{/number}","milestones_url":"https://api.github.com/repos/mozilla/zamboni/milestones{/number}","notifications_url":"https://api.github.com/repos/mozilla/zamboni/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/mozilla/zamboni/labels{/name}","releases_url":"https://api.github.com/repos/mozilla/zamboni/releases{/id}","created_at":"2009-10-23T19:35:34Z","updated_at":"2014-09-02T16:58:56Z","pushed_at":"2014-09-03T18:05:56Z","git_url":"git://github.com/mozilla/zamboni.git","ssh_url":"git@github.com:mozilla/zamboni.git","clone_url":"https://github.com/mozilla/zamboni.git","svn_url":"https://github.com/mozilla/zamboni","homepage":"http://zamboni.readthedocs.org","size":360855,"stargazers_count":430,"watchers_count":430,"language":"Python","has_issues":false,"has_downloads":false,"has_wiki":false,"forks_count":232,"mirror_url":null,"open_issues_count":16,"forks":232,"open_issues":16,"watchers":430,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/mozilla/zamboni/pulls/2496"},"html":{"href":"https://github.com/mozilla/zamboni/pull/2496"},"issue":{"href":"https://api.github.com/repos/mozilla/zamboni/issues/2496"},"comments":{"href":"https://api.github.com/repos/mozilla/zamboni/issues/2496/comments"},"review_comments":{"href":"https://api.github.com/repos/mozilla/zamboni/pulls/2496/comments"},"review_comment":{"href":"https://api.github.com/repos/mozilla/zamboni/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/mozilla/zamboni/pulls/2496/commits"},"statuses":{"href":"https://api.github.com/repos/mozilla/zamboni/statuses/bcefcaec4d1d29405a2b074da0b110c296d04a83"}}}}