{"action":"created","issue":{"url":"https://api.github.com/repos/plone/Products.CMFPlone/issues/1068","labels_url":"https://api.github.com/repos/plone/Products.CMFPlone/issues/1068/labels{/name}","comments_url":"https://api.github.com/repos/plone/Products.CMFPlone/issues/1068/comments","events_url":"https://api.github.com/repos/plone/Products.CMFPlone/issues/1068/events","html_url":"https://github.com/plone/Products.CMFPlone/pull/1068","id":107962722,"number":1068,"title":"Fix #1067: mail controlpanel not keeping password field when saving","user":{"login":"allusa","id":10347907,"avatar_url":"https://avatars.githubusercontent.com/u/10347907?v=3","gravatar_id":"","url":"https://api.github.com/users/allusa","html_url":"https://github.com/allusa","followers_url":"https://api.github.com/users/allusa/followers","following_url":"https://api.github.com/users/allusa/following{/other_user}","gists_url":"https://api.github.com/users/allusa/gists{/gist_id}","starred_url":"https://api.github.com/users/allusa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/allusa/subscriptions","organizations_url":"https://api.github.com/users/allusa/orgs","repos_url":"https://api.github.com/users/allusa/repos","events_url":"https://api.github.com/users/allusa/events{/privacy}","received_events_url":"https://api.github.com/users/allusa/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/plone/Products.CMFPlone/labels/pr%20review","name":"pr review","color":"fbca04"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-09-23T16:59:13Z","updated_at":"2015-09-24T07:25:17Z","closed_at":"2015-09-24T07:14:12Z","pull_request":{"url":"https://api.github.com/repos/plone/Products.CMFPlone/pulls/1068","html_url":"https://github.com/plone/Products.CMFPlone/pull/1068","diff_url":"https://github.com/plone/Products.CMFPlone/pull/1068.diff","patch_url":"https://github.com/plone/Products.CMFPlone/pull/1068.patch"},"body":"ESMTP password is not shown in controlpanel, thus it is returned as None when saving #1067\r\n@bloodbare "},"comment":{"url":"https://api.github.com/repos/plone/Products.CMFPlone/issues/comments/142837978","html_url":"https://github.com/plone/Products.CMFPlone/pull/1068#issuecomment-142837978","issue_url":"https://api.github.com/repos/plone/Products.CMFPlone/issues/1068","id":142837978,"user":{"login":"tisto","id":632105,"avatar_url":"https://avatars.githubusercontent.com/u/632105?v=3","gravatar_id":"","url":"https://api.github.com/users/tisto","html_url":"https://github.com/tisto","followers_url":"https://api.github.com/users/tisto/followers","following_url":"https://api.github.com/users/tisto/following{/other_user}","gists_url":"https://api.github.com/users/tisto/gists{/gist_id}","starred_url":"https://api.github.com/users/tisto/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tisto/subscriptions","organizations_url":"https://api.github.com/users/tisto/orgs","repos_url":"https://api.github.com/users/tisto/repos","events_url":"https://api.github.com/users/tisto/events{/privacy}","received_events_url":"https://api.github.com/users/tisto/received_events","type":"User","site_admin":false},"created_at":"2015-09-24T07:25:17Z","updated_at":"2015-09-24T07:25:17Z","body":"@jensens I intentionally did not merge that pr because I think we should take tests seriously. You saw that and decided to merge anyways. That's pretty rude IMHO.\r\n\r\nI put a lot of effort into writing a lot of tests for this feature and it is really easy to write this particular test (maybe two lines of additional code). It would take me two minutes to write that test, but I wanted @allcaps to give it a try. \r\n\r\nMaybe I just ended up in the wrong community, if tests that take a few minutes to write are preventing us \"to get things done\"..."}}