{"action":"created","comment":{"url":"https://api.github.com/repos/Havven/havven/pulls/comments/234091633","pull_request_review_id":175659903,"id":234091633,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDIzNDA5MTYzMw==","diff_hunk":"@@ -269,8 +293,15 @@ contract Depot is SafeDecimalMath, SelfDestructible, Pausable {\n                     deposit.amount = safeSub(deposit.amount, remainingToFulfill);\n                     totalSellableDeposits = safeSub(totalSellableDeposits, remainingToFulfill);\n \n-                    // Transfer the ETH to the depositor.\n-                    deposit.user.transfer(safeDiv_dec(remainingToFulfill, usdToEthPrice));\n+                    // Transfer the ETH to the depositor. Send is used instead of transfer\n+                    // so a non payable contract wont block the FIFO queue on a failed","path":"contracts/Depot.sol","position":null,"original_position":61,"commit_id":"74e9813682ab18c0a17c116327c6a7a176582fe4","original_commit_id":"8c972f807c44c60a5fdd3da512728130739ca067","user":{"login":"hav-noms","id":1251099,"node_id":"MDQ6VXNlcjEyNTEwOTk=","avatar_url":"https://avatars2.githubusercontent.com/u/1251099?v=4","gravatar_id":"","url":"https://api.github.com/users/hav-noms","html_url":"https://github.com/hav-noms","followers_url":"https://api.github.com/users/hav-noms/followers","following_url":"https://api.github.com/users/hav-noms/following{/other_user}","gists_url":"https://api.github.com/users/hav-noms/gists{/gist_id}","starred_url":"https://api.github.com/users/hav-noms/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hav-noms/subscriptions","organizations_url":"https://api.github.com/users/hav-noms/orgs","repos_url":"https://api.github.com/users/hav-noms/repos","events_url":"https://api.github.com/users/hav-noms/events{/privacy}","received_events_url":"https://api.github.com/users/hav-noms/received_events","type":"User","site_admin":false},"body":"fixed","created_at":"2018-11-16T05:11:31Z","updated_at":"2018-11-16T05:11:31Z","html_url":"https://github.com/Havven/havven/pull/81#discussion_r234091633","pull_request_url":"https://api.github.com/repos/Havven/havven/pulls/81","author_association":"MEMBER","_links":{"self":{"href":"https://api.github.com/repos/Havven/havven/pulls/comments/234091633"},"html":{"href":"https://github.com/Havven/havven/pull/81#discussion_r234091633"},"pull_request":{"href":"https://api.github.com/repos/Havven/havven/pulls/81"}},"in_reply_to_id":234085592},"pull_request":{"url":"https://api.github.com/repos/Havven/havven/pulls/81","id":227075615,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI3MDc1NjE1","html_url":"https://github.com/Havven/havven/pull/81","diff_url":"https://github.com/Havven/havven/pull/81.diff","patch_url":"https://github.com/Havven/havven/pull/81.patch","issue_url":"https://api.github.com/repos/Havven/havven/issues/81","number":81,"state":"open","locked":false,"title":"Depot audit remediation","user":{"login":"hav-noms","id":1251099,"node_id":"MDQ6VXNlcjEyNTEwOTk=","avatar_url":"https://avatars2.githubusercontent.com/u/1251099?v=4","gravatar_id":"","url":"https://api.github.com/users/hav-noms","html_url":"https://github.com/hav-noms","followers_url":"https://api.github.com/users/hav-noms/followers","following_url":"https://api.github.com/users/hav-noms/following{/other_user}","gists_url":"https://api.github.com/users/hav-noms/gists{/gist_id}","starred_url":"https://api.github.com/users/hav-noms/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hav-noms/subscriptions","organizations_url":"https://api.github.com/users/hav-noms/orgs","repos_url":"https://api.github.com/users/hav-noms/repos","events_url":"https://api.github.com/users/hav-noms/events{/privacy}","received_events_url":"https://api.github.com/users/hav-noms/received_events","type":"User","site_admin":false},"body":"","created_at":"2018-10-30T20:46:44Z","updated_at":"2018-11-16T05:11:31Z","closed_at":null,"merged_at":null,"merge_commit_sha":"933813173cf4e535008d59443ecac320307b7792","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/Havven/havven/pulls/81/commits","review_comments_url":"https://api.github.com/repos/Havven/havven/pulls/81/comments","review_comment_url":"https://api.github.com/repos/Havven/havven/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Havven/havven/issues/81/comments","statuses_url":"https://api.github.com/repos/Havven/havven/statuses/74e9813682ab18c0a17c116327c6a7a176582fe4","head":{"label":"Havven:depot-audit-remediation","ref":"depot-audit-remediation","sha":"74e9813682ab18c0a17c116327c6a7a176582fe4","user":{"login":"Havven","id":30892445,"node_id":"MDEyOk9yZ2FuaXphdGlvbjMwODkyNDQ1","avatar_url":"https://avatars3.githubusercontent.com/u/30892445?v=4","gravatar_id":"","url":"https://api.github.com/users/Havven","html_url":"https://github.com/Havven","followers_url":"https://api.github.com/users/Havven/followers","following_url":"https://api.github.com/users/Havven/following{/other_user}","gists_url":"https://api.github.com/users/Havven/gists{/gist_id}","starred_url":"https://api.github.com/users/Havven/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Havven/subscriptions","organizations_url":"https://api.github.com/users/Havven/orgs","repos_url":"https://api.github.com/users/Havven/repos","events_url":"https://api.github.com/users/Havven/events{/privacy}","received_events_url":"https://api.github.com/users/Havven/received_events","type":"Organization","site_admin":false},"repo":{"id":112581082,"node_id":"MDEwOlJlcG9zaXRvcnkxMTI1ODEwODI=","name":"havven","full_name":"Havven/havven","private":false,"owner":{"login":"Havven","id":30892445,"node_id":"MDEyOk9yZ2FuaXphdGlvbjMwODkyNDQ1","avatar_url":"https://avatars3.githubusercontent.com/u/30892445?v=4","gravatar_id":"","url":"https://api.github.com/users/Havven","html_url":"https://github.com/Havven","followers_url":"https://api.github.com/users/Havven/followers","following_url":"https://api.github.com/users/Havven/following{/other_user}","gists_url":"https://api.github.com/users/Havven/gists{/gist_id}","starred_url":"https://api.github.com/users/Havven/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Havven/subscriptions","organizations_url":"https://api.github.com/users/Havven/orgs","repos_url":"https://api.github.com/users/Havven/repos","events_url":"https://api.github.com/users/Havven/events{/privacy}","received_events_url":"https://api.github.com/users/Havven/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/Havven/havven","description":"Havven Solidity smart contracts","fork":false,"url":"https://api.github.com/repos/Havven/havven","forks_url":"https://api.github.com/repos/Havven/havven/forks","keys_url":"https://api.github.com/repos/Havven/havven/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Havven/havven/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Havven/havven/teams","hooks_url":"https://api.github.com/repos/Havven/havven/hooks","issue_events_url":"https://api.github.com/repos/Havven/havven/issues/events{/number}","events_url":"https://api.github.com/repos/Havven/havven/events","assignees_url":"https://api.github.com/repos/Havven/havven/assignees{/user}","branches_url":"https://api.github.com/repos/Havven/havven/branches{/branch}","tags_url":"https://api.github.com/repos/Havven/havven/tags","blobs_url":"https://api.github.com/repos/Havven/havven/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Havven/havven/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Havven/havven/git/refs{/sha}","trees_url":"https://api.github.com/repos/Havven/havven/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Havven/havven/statuses/{sha}","languages_url":"https://api.github.com/repos/Havven/havven/languages","stargazers_url":"https://api.github.com/repos/Havven/havven/stargazers","contributors_url":"https://api.github.com/repos/Havven/havven/contributors","subscribers_url":"https://api.github.com/repos/Havven/havven/subscribers","subscription_url":"https://api.github.com/repos/Havven/havven/subscription","commits_url":"https://api.github.com/repos/Havven/havven/commits{/sha}","git_commits_url":"https://api.github.com/repos/Havven/havven/git/commits{/sha}","comments_url":"https://api.github.com/repos/Havven/havven/comments{/number}","issue_comment_url":"https://api.github.com/repos/Havven/havven/issues/comments{/number}","contents_url":"https://api.github.com/repos/Havven/havven/contents/{+path}","compare_url":"https://api.github.com/repos/Havven/havven/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Havven/havven/merges","archive_url":"https://api.github.com/repos/Havven/havven/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Havven/havven/downloads","issues_url":"https://api.github.com/repos/Havven/havven/issues{/number}","pulls_url":"https://api.github.com/repos/Havven/havven/pulls{/number}","milestones_url":"https://api.github.com/repos/Havven/havven/milestones{/number}","notifications_url":"https://api.github.com/repos/Havven/havven/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Havven/havven/labels{/name}","releases_url":"https://api.github.com/repos/Havven/havven/releases{/id}","deployments_url":"https://api.github.com/repos/Havven/havven/deployments","created_at":"2017-11-30T07:43:43Z","updated_at":"2018-11-12T20:09:13Z","pushed_at":"2018-11-16T05:07:26Z","git_url":"git://github.com/Havven/havven.git","ssh_url":"git@github.com:Havven/havven.git","clone_url":"https://github.com/Havven/havven.git","svn_url":"https://github.com/Havven/havven","homepage":"https://havven.io/","size":2311,"stargazers_count":62,"watchers_count":62,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":16,"mirror_url":null,"archived":false,"open_issues_count":2,"license":{"key":"mit","name":"MIT License","spdx_id":"MIT","url":"https://api.github.com/licenses/mit","node_id":"MDc6TGljZW5zZTEz"},"forks":16,"open_issues":2,"watchers":62,"default_branch":"master"}},"base":{"label":"Havven:truffle-impl-v2","ref":"truffle-impl-v2","sha":"481c88d047bfb7b1f1c3f17f06e22dbf5e5b0878","user":{"login":"Havven","id":30892445,"node_id":"MDEyOk9yZ2FuaXphdGlvbjMwODkyNDQ1","avatar_url":"https://avatars3.githubusercontent.com/u/30892445?v=4","gravatar_id":"","url":"https://api.github.com/users/Havven","html_url":"https://github.com/Havven","followers_url":"https://api.github.com/users/Havven/followers","following_url":"https://api.github.com/users/Havven/following{/other_user}","gists_url":"https://api.github.com/users/Havven/gists{/gist_id}","starred_url":"https://api.github.com/users/Havven/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Havven/subscriptions","organizations_url":"https://api.github.com/users/Havven/orgs","repos_url":"https://api.github.com/users/Havven/repos","events_url":"https://api.github.com/users/Havven/events{/privacy}","received_events_url":"https://api.github.com/users/Havven/received_events","type":"Organization","site_admin":false},"repo":{"id":112581082,"node_id":"MDEwOlJlcG9zaXRvcnkxMTI1ODEwODI=","name":"havven","full_name":"Havven/havven","private":false,"owner":{"login":"Havven","id":30892445,"node_id":"MDEyOk9yZ2FuaXphdGlvbjMwODkyNDQ1","avatar_url":"https://avatars3.githubusercontent.com/u/30892445?v=4","gravatar_id":"","url":"https://api.github.com/users/Havven","html_url":"https://github.com/Havven","followers_url":"https://api.github.com/users/Havven/followers","following_url":"https://api.github.com/users/Havven/following{/other_user}","gists_url":"https://api.github.com/users/Havven/gists{/gist_id}","starred_url":"https://api.github.com/users/Havven/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Havven/subscriptions","organizations_url":"https://api.github.com/users/Havven/orgs","repos_url":"https://api.github.com/users/Havven/repos","events_url":"https://api.github.com/users/Havven/events{/privacy}","received_events_url":"https://api.github.com/users/Havven/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/Havven/havven","description":"Havven Solidity smart contracts","fork":false,"url":"https://api.github.com/repos/Havven/havven","forks_url":"https://api.github.com/repos/Havven/havven/forks","keys_url":"https://api.github.com/repos/Havven/havven/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Havven/havven/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Havven/havven/teams","hooks_url":"https://api.github.com/repos/Havven/havven/hooks","issue_events_url":"https://api.github.com/repos/Havven/havven/issues/events{/number}","events_url":"https://api.github.com/repos/Havven/havven/events","assignees_url":"https://api.github.com/repos/Havven/havven/assignees{/user}","branches_url":"https://api.github.com/repos/Havven/havven/branches{/branch}","tags_url":"https://api.github.com/repos/Havven/havven/tags","blobs_url":"https://api.github.com/repos/Havven/havven/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Havven/havven/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Havven/havven/git/refs{/sha}","trees_url":"https://api.github.com/repos/Havven/havven/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Havven/havven/statuses/{sha}","languages_url":"https://api.github.com/repos/Havven/havven/languages","stargazers_url":"https://api.github.com/repos/Havven/havven/stargazers","contributors_url":"https://api.github.com/repos/Havven/havven/contributors","subscribers_url":"https://api.github.com/repos/Havven/havven/subscribers","subscription_url":"https://api.github.com/repos/Havven/havven/subscription","commits_url":"https://api.github.com/repos/Havven/havven/commits{/sha}","git_commits_url":"https://api.github.com/repos/Havven/havven/git/commits{/sha}","comments_url":"https://api.github.com/repos/Havven/havven/comments{/number}","issue_comment_url":"https://api.github.com/repos/Havven/havven/issues/comments{/number}","contents_url":"https://api.github.com/repos/Havven/havven/contents/{+path}","compare_url":"https://api.github.com/repos/Havven/havven/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Havven/havven/merges","archive_url":"https://api.github.com/repos/Havven/havven/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Havven/havven/downloads","issues_url":"https://api.github.com/repos/Havven/havven/issues{/number}","pulls_url":"https://api.github.com/repos/Havven/havven/pulls{/number}","milestones_url":"https://api.github.com/repos/Havven/havven/milestones{/number}","notifications_url":"https://api.github.com/repos/Havven/havven/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Havven/havven/labels{/name}","releases_url":"https://api.github.com/repos/Havven/havven/releases{/id}","deployments_url":"https://api.github.com/repos/Havven/havven/deployments","created_at":"2017-11-30T07:43:43Z","updated_at":"2018-11-12T20:09:13Z","pushed_at":"2018-11-16T05:07:26Z","git_url":"git://github.com/Havven/havven.git","ssh_url":"git@github.com:Havven/havven.git","clone_url":"https://github.com/Havven/havven.git","svn_url":"https://github.com/Havven/havven","homepage":"https://havven.io/","size":2311,"stargazers_count":62,"watchers_count":62,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":16,"mirror_url":null,"archived":false,"open_issues_count":2,"license":{"key":"mit","name":"MIT License","spdx_id":"MIT","url":"https://api.github.com/licenses/mit","node_id":"MDc6TGljZW5zZTEz"},"forks":16,"open_issues":2,"watchers":62,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Havven/havven/pulls/81"},"html":{"href":"https://github.com/Havven/havven/pull/81"},"issue":{"href":"https://api.github.com/repos/Havven/havven/issues/81"},"comments":{"href":"https://api.github.com/repos/Havven/havven/issues/81/comments"},"review_comments":{"href":"https://api.github.com/repos/Havven/havven/pulls/81/comments"},"review_comment":{"href":"https://api.github.com/repos/Havven/havven/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Havven/havven/pulls/81/commits"},"statuses":{"href":"https://api.github.com/repos/Havven/havven/statuses/74e9813682ab18c0a17c116327c6a7a176582fe4"}},"author_association":"MEMBER"}}
{"actor":{"display_login":"hav-noms"}}