{"action":"created","issue":{"url":"https://api.github.com/repos/mozilla/addons-server/issues/9353","repository_url":"https://api.github.com/repos/mozilla/addons-server","labels_url":"https://api.github.com/repos/mozilla/addons-server/issues/9353/labels{/name}","comments_url":"https://api.github.com/repos/mozilla/addons-server/issues/9353/comments","events_url":"https://api.github.com/repos/mozilla/addons-server/issues/9353/events","html_url":"https://github.com/mozilla/addons-server/pull/9353","id":357279551,"node_id":"MDExOlB1bGxSZXF1ZXN0MjEzMzMzMTQw","number":9353,"title":"Add Addon submission reporting to Akismet","user":{"login":"eviljeff","id":768592,"node_id":"MDQ6VXNlcjc2ODU5Mg==","avatar_url":"https://avatars2.githubusercontent.com/u/768592?v=4","gravatar_id":"","url":"https://api.github.com/users/eviljeff","html_url":"https://github.com/eviljeff","followers_url":"https://api.github.com/users/eviljeff/followers","following_url":"https://api.github.com/users/eviljeff/following{/other_user}","gists_url":"https://api.github.com/users/eviljeff/gists{/gist_id}","starred_url":"https://api.github.com/users/eviljeff/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eviljeff/subscriptions","organizations_url":"https://api.github.com/users/eviljeff/orgs","repos_url":"https://api.github.com/users/eviljeff/repos","events_url":"https://api.github.com/users/eviljeff/events{/privacy}","received_events_url":"https://api.github.com/users/eviljeff/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2018-09-05T15:19:02Z","updated_at":"2018-09-17T15:36:44Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/mozilla/addons-server/pulls/9353","html_url":"https://github.com/mozilla/addons-server/pull/9353","diff_url":"https://github.com/mozilla/addons-server/pull/9353.diff","patch_url":"https://github.com/mozilla/addons-server/pull/9353.patch"},"body":"First part of #8704 - any new addon or version will be spam checked with Akismet. Second part will be handling post submission edits (both in the update flow, i.e. details step, and in edit listing afterwards).  This patch was already getting to an unmanageable size so thought I'd submit this part first."},"comment":{"url":"https://api.github.com/repos/mozilla/addons-server/issues/comments/422065281","html_url":"https://github.com/mozilla/addons-server/pull/9353#issuecomment-422065281","issue_url":"https://api.github.com/repos/mozilla/addons-server/issues/9353","id":422065281,"node_id":"MDEyOklzc3VlQ29tbWVudDQyMjA2NTI4MQ==","user":{"login":"eviljeff","id":768592,"node_id":"MDQ6VXNlcjc2ODU5Mg==","avatar_url":"https://avatars2.githubusercontent.com/u/768592?v=4","gravatar_id":"","url":"https://api.github.com/users/eviljeff","html_url":"https://github.com/eviljeff","followers_url":"https://api.github.com/users/eviljeff/followers","following_url":"https://api.github.com/users/eviljeff/following{/other_user}","gists_url":"https://api.github.com/users/eviljeff/gists{/gist_id}","starred_url":"https://api.github.com/users/eviljeff/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eviljeff/subscriptions","organizations_url":"https://api.github.com/users/eviljeff/orgs","repos_url":"https://api.github.com/users/eviljeff/repos","events_url":"https://api.github.com/users/eviljeff/events{/privacy}","received_events_url":"https://api.github.com/users/eviljeff/received_events","type":"User","site_admin":false},"created_at":"2018-09-17T15:36:44Z","updated_at":"2018-09-17T15:36:44Z","author_association":"MEMBER","body":"> Can we put this into `handle_upload_validation_result` instead maybe if it has to be done during validation? That's already used for a lot of validation-result annotation and feels like a better place to me.\r\n\r\nI did have it there on Monday/Tuesday (I didn't keep the commit unfortunately) but we either have to delaying saving _all_ the results to FileUpload until after Akismet has finished the `comment_check`s or accept the edge case that `FileUpload.valid` will be `true` before the Akismet checks have finished and we call `save()` again.  (calling `FileUpload.save` sets `.valid` if any validation results exist)"}}
{"actor":{"display_login":"eviljeff"}}