{"action":"created","issue":{"url":"https://api.github.com/repos/tl-its-umich-edu/ctools-project-migration/issues/324","repository_url":"https://api.github.com/repos/tl-its-umich-edu/ctools-project-migration","labels_url":"https://api.github.com/repos/tl-its-umich-edu/ctools-project-migration/issues/324/labels{/name}","comments_url":"https://api.github.com/repos/tl-its-umich-edu/ctools-project-migration/issues/324/comments","events_url":"https://api.github.com/repos/tl-its-umich-edu/ctools-project-migration/issues/324/events","html_url":"https://github.com/tl-its-umich-edu/ctools-project-migration/pull/324","id":186526762,"number":324,"title":"Tlcpm 451 fixing the 500 error caused when getting Google Group Id","user":{"login":"pushyamig","id":8579775,"avatar_url":"https://avatars.githubusercontent.com/u/8579775?v=3","gravatar_id":"","url":"https://api.github.com/users/pushyamig","html_url":"https://github.com/pushyamig","followers_url":"https://api.github.com/users/pushyamig/followers","following_url":"https://api.github.com/users/pushyamig/following{/other_user}","gists_url":"https://api.github.com/users/pushyamig/gists{/gist_id}","starred_url":"https://api.github.com/users/pushyamig/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pushyamig/subscriptions","organizations_url":"https://api.github.com/users/pushyamig/orgs","repos_url":"https://api.github.com/users/pushyamig/repos","events_url":"https://api.github.com/users/pushyamig/events{/privacy}","received_events_url":"https://api.github.com/users/pushyamig/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-11-01T12:52:24Z","updated_at":"2016-11-01T21:02:33Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/tl-its-umich-edu/ctools-project-migration/pulls/324","html_url":"https://github.com/tl-its-umich-edu/ctools-project-migration/pull/324","diff_url":"https://github.com/tl-its-umich-edu/ctools-project-migration/pull/324.diff","patch_url":"https://github.com/tl-its-umich-edu/ctools-project-migration/pull/324.patch"},"body":"@dlhaines "},"comment":{"url":"https://api.github.com/repos/tl-its-umich-edu/ctools-project-migration/issues/comments/257696026","html_url":"https://github.com/tl-its-umich-edu/ctools-project-migration/pull/324#issuecomment-257696026","issue_url":"https://api.github.com/repos/tl-its-umich-edu/ctools-project-migration/issues/324","id":257696026,"user":{"login":"dlhaines","id":74063,"avatar_url":"https://avatars.githubusercontent.com/u/74063?v=3","gravatar_id":"","url":"https://api.github.com/users/dlhaines","html_url":"https://github.com/dlhaines","followers_url":"https://api.github.com/users/dlhaines/followers","following_url":"https://api.github.com/users/dlhaines/following{/other_user}","gists_url":"https://api.github.com/users/dlhaines/gists{/gist_id}","starred_url":"https://api.github.com/users/dlhaines/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dlhaines/subscriptions","organizations_url":"https://api.github.com/users/dlhaines/orgs","repos_url":"https://api.github.com/users/dlhaines/repos","events_url":"https://api.github.com/users/dlhaines/events{/privacy}","received_events_url":"https://api.github.com/users/dlhaines/received_events","type":"User","site_admin":false},"created_at":"2016-11-01T21:02:33Z","updated_at":"2016-11-01T21:02:33Z","body":"I think this is fine except that the unit tests MUST be updated before the check-in is acceptable.  Letting unit tests get out of date makes the tests much less useful.  That just kicks work down the road.  The next person who comes along can't trust that the unit test are good so they either have to fix them or they get to make all the mistakes that good unit tests help people avoid.  (And no matter how many times you get told any piece of code is dead end code it isn't.)\r\n\r\nIt would also be good to catch the exception close to this code so that we can immediately log any email unexpected troublesome addresses that give us problems in the future."}}
{"actor":{"display_login":"dlhaines"}}