{"action":"created","issue":{"url":"https://api.github.com/repos/hapijs/catbox-mongodb/issues/28","repository_url":"https://api.github.com/repos/hapijs/catbox-mongodb","labels_url":"https://api.github.com/repos/hapijs/catbox-mongodb/issues/28/labels{/name}","comments_url":"https://api.github.com/repos/hapijs/catbox-mongodb/issues/28/comments","events_url":"https://api.github.com/repos/hapijs/catbox-mongodb/issues/28/events","html_url":"https://github.com/hapijs/catbox-mongodb/pull/28","id":272182820,"number":28,"title":"Update to catbox 10","user":{"login":"marcuspoehls","id":837771,"avatar_url":"https://avatars1.githubusercontent.com/u/837771?v=4","gravatar_id":"","url":"https://api.github.com/users/marcuspoehls","html_url":"https://github.com/marcuspoehls","followers_url":"https://api.github.com/users/marcuspoehls/followers","following_url":"https://api.github.com/users/marcuspoehls/following{/other_user}","gists_url":"https://api.github.com/users/marcuspoehls/gists{/gist_id}","starred_url":"https://api.github.com/users/marcuspoehls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marcuspoehls/subscriptions","organizations_url":"https://api.github.com/users/marcuspoehls/orgs","repos_url":"https://api.github.com/users/marcuspoehls/repos","events_url":"https://api.github.com/users/marcuspoehls/events{/privacy}","received_events_url":"https://api.github.com/users/marcuspoehls/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2017-11-08T12:35:05Z","updated_at":"2017-11-14T11:00:49Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/hapijs/catbox-mongodb/pulls/28","html_url":"https://github.com/hapijs/catbox-mongodb/pull/28","diff_url":"https://github.com/hapijs/catbox-mongodb/pull/28.diff","patch_url":"https://github.com/hapijs/catbox-mongodb/pull/28.patch"},"body":"This PR includes the following updates\r\n\r\n- support for `catbox` 10\r\n- refactor library to use an ES2015 `class`\r\n- use `async/await` with promises from MongoDB's lib\r\n- update tests to support changed library code\r\n- bump dependencies to their latest versions\r\n- requires Node.js `8.9.0` or later, defined in `package.json`\r\n- update `.travis.yml` to test Node.js versions 8 and 9\r\n\r\nSolves https://github.com/hapijs/catbox-mongodb/issues/27 (catbox 10 support)"},"comment":{"url":"https://api.github.com/repos/hapijs/catbox-mongodb/issues/comments/344223200","html_url":"https://github.com/hapijs/catbox-mongodb/pull/28#issuecomment-344223200","issue_url":"https://api.github.com/repos/hapijs/catbox-mongodb/issues/28","id":344223200,"user":{"login":"jardakotesovec","id":1590681,"avatar_url":"https://avatars3.githubusercontent.com/u/1590681?v=4","gravatar_id":"","url":"https://api.github.com/users/jardakotesovec","html_url":"https://github.com/jardakotesovec","followers_url":"https://api.github.com/users/jardakotesovec/followers","following_url":"https://api.github.com/users/jardakotesovec/following{/other_user}","gists_url":"https://api.github.com/users/jardakotesovec/gists{/gist_id}","starred_url":"https://api.github.com/users/jardakotesovec/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jardakotesovec/subscriptions","organizations_url":"https://api.github.com/users/jardakotesovec/orgs","repos_url":"https://api.github.com/users/jardakotesovec/repos","events_url":"https://api.github.com/users/jardakotesovec/events{/privacy}","received_events_url":"https://api.github.com/users/jardakotesovec/received_events","type":"User","site_admin":false},"created_at":"2017-11-14T11:00:49Z","updated_at":"2017-11-14T11:00:49Z","author_association":"MEMBER","body":"@marcuspoehls Thanks for updates, looks good. I also learned something about await and async :-).\r\n\r\nI want to do one more detailed review to make sure I am leaving this module at least at the same shape as I got it :-). I will get on it likely on Sunday.\r\n\r\nI suspect that whats not handled in compared to original implementation is that if the connection fails - you can't really call start again - you will keep returning rejected promise instead of start connecting again. In original implementation it would start connection again I believe.  Have not checked that in detail..  Would be cool to cover that with test."}}
{"actor":{"display_login":"jardakotesovec"}}