{"action":"created","issue":{"url":"https://api.github.com/repos/bitcoin/bitcoin/issues/6134","labels_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/6134/labels{/name}","comments_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/6134/comments","events_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/6134/events","html_url":"https://github.com/bitcoin/bitcoin/pull/6134","id":76116300,"number":6134,"title":"Improve usage of fee estimation code","user":{"login":"morcos","id":4360349,"avatar_url":"https://avatars.githubusercontent.com/u/4360349?v=3","gravatar_id":"","url":"https://api.github.com/users/morcos","html_url":"https://github.com/morcos","followers_url":"https://api.github.com/users/morcos/followers","following_url":"https://api.github.com/users/morcos/following{/other_user}","gists_url":"https://api.github.com/users/morcos/gists{/gist_id}","starred_url":"https://api.github.com/users/morcos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/morcos/subscriptions","organizations_url":"https://api.github.com/users/morcos/orgs","repos_url":"https://api.github.com/users/morcos/repos","events_url":"https://api.github.com/users/morcos/events{/privacy}","received_events_url":"https://api.github.com/users/morcos/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/bitcoin/bitcoin/labels/Wallet","name":"Wallet","color":"02d7e1"}],"state":"closed","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/bitcoin/bitcoin/milestones/19","html_url":"https://github.com/bitcoin/bitcoin/milestones/0.12.0","labels_url":"https://api.github.com/repos/bitcoin/bitcoin/milestones/19/labels","id":1092458,"number":19,"title":"0.12.0","description":"","creator":{"login":"laanwj","id":126646,"avatar_url":"https://avatars.githubusercontent.com/u/126646?v=3","gravatar_id":"","url":"https://api.github.com/users/laanwj","html_url":"https://github.com/laanwj","followers_url":"https://api.github.com/users/laanwj/followers","following_url":"https://api.github.com/users/laanwj/following{/other_user}","gists_url":"https://api.github.com/users/laanwj/gists{/gist_id}","starred_url":"https://api.github.com/users/laanwj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/laanwj/subscriptions","organizations_url":"https://api.github.com/users/laanwj/orgs","repos_url":"https://api.github.com/users/laanwj/repos","events_url":"https://api.github.com/users/laanwj/events{/privacy}","received_events_url":"https://api.github.com/users/laanwj/received_events","type":"User","site_admin":false},"open_issues":9,"closed_issues":24,"state":"open","created_at":"2015-05-01T12:31:35Z","updated_at":"2015-11-27T12:50:21Z","due_on":null,"closed_at":null},"comments":17,"created_at":"2015-05-13T20:39:41Z","updated_at":"2015-11-27T14:04:55Z","closed_at":"2015-11-27T12:04:59Z","pull_request":{"url":"https://api.github.com/repos/bitcoin/bitcoin/pulls/6134","html_url":"https://github.com/bitcoin/bitcoin/pull/6134","diff_url":"https://github.com/bitcoin/bitcoin/pull/6134.diff","patch_url":"https://github.com/bitcoin/bitcoin/pull/6134.patch"},"body":"When automatically adding a fee estimate to a transaction and in the event no estimate is available for the desired confirmation target, it makes more sense to default to a fee estimate for a worse confirmation target rather than the hard coded minimum.\r\n\r\nI'm open to a different way of exposing this functionality if there is a better suggestion.\r\n\r\n@cozz I don't know anything about QT code, so I think what I did in sendcoinsdialog.cpp is pretty hacky, although it appears to have the desired effect. Perhaps you could take a look and suggest a better approach? This is my attempt to address the concern I mentioned in this comment: https://github.com/bitcoin/bitcoin/pull/5200#issuecomment-61545304"},"comment":{"url":"https://api.github.com/repos/bitcoin/bitcoin/issues/comments/160146271","html_url":"https://github.com/bitcoin/bitcoin/pull/6134#issuecomment-160146271","issue_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/6134","id":160146271,"user":{"login":"jtimon","id":1008458,"avatar_url":"https://avatars.githubusercontent.com/u/1008458?v=3","gravatar_id":"","url":"https://api.github.com/users/jtimon","html_url":"https://github.com/jtimon","followers_url":"https://api.github.com/users/jtimon/followers","following_url":"https://api.github.com/users/jtimon/following{/other_user}","gists_url":"https://api.github.com/users/jtimon/gists{/gist_id}","starred_url":"https://api.github.com/users/jtimon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jtimon/subscriptions","organizations_url":"https://api.github.com/users/jtimon/orgs","repos_url":"https://api.github.com/users/jtimon/repos","events_url":"https://api.github.com/users/jtimon/events{/privacy}","received_events_url":"https://api.github.com/users/jtimon/received_events","type":"User","site_admin":false},"created_at":"2015-11-27T14:04:55Z","updated_at":"2015-11-27T14:04:55Z","body":"Opened #7115 to fix the new circular dependency introduced despite my insistence and despite coding the commit to be squashed here to avoid that circular dependency..."}}