{"action":"created","issue":{"url":"https://api.github.com/repos/wordpress-mobile/WordPress-iOS/issues/4537","labels_url":"https://api.github.com/repos/wordpress-mobile/WordPress-iOS/issues/4537/labels{/name}","comments_url":"https://api.github.com/repos/wordpress-mobile/WordPress-iOS/issues/4537/comments","events_url":"https://api.github.com/repos/wordpress-mobile/WordPress-iOS/issues/4537/events","html_url":"https://github.com/wordpress-mobile/WordPress-iOS/pull/4537","id":119823968,"number":4537,"title":"Nil checks for preventing crash when using dotComIDs for Tracks.","user":{"login":"kurzee","id":1873422,"avatar_url":"https://avatars.githubusercontent.com/u/1873422?v=3","gravatar_id":"","url":"https://api.github.com/users/kurzee","html_url":"https://github.com/kurzee","followers_url":"https://api.github.com/users/kurzee/followers","following_url":"https://api.github.com/users/kurzee/following{/other_user}","gists_url":"https://api.github.com/users/kurzee/gists{/gist_id}","starred_url":"https://api.github.com/users/kurzee/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kurzee/subscriptions","organizations_url":"https://api.github.com/users/kurzee/orgs","repos_url":"https://api.github.com/users/kurzee/repos","events_url":"https://api.github.com/users/kurzee/events{/privacy}","received_events_url":"https://api.github.com/users/kurzee/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-12-01T22:35:06Z","updated_at":"2015-12-02T09:28:24Z","closed_at":"2015-12-02T00:12:01Z","pull_request":{"url":"https://api.github.com/repos/wordpress-mobile/WordPress-iOS/pulls/4537","html_url":"https://github.com/wordpress-mobile/WordPress-iOS/pull/4537","diff_url":"https://github.com/wordpress-mobile/WordPress-iOS/pull/4537.diff","patch_url":"https://github.com/wordpress-mobile/WordPress-iOS/pull/4537.patch"},"body":"Changes in https://github.com/wordpress-mobile/WordPress-iOS/commit/0b7e2da6103874db4da49dda77f35321310db38a created multiple crash cases for when the `dotComID` method returns `nil`.\r\n\r\nTesting for crash:\r\n1. Open app\r\n2. Hit Post button on TabBar (crash)\r\n3. Dismiss TabBar (crash)\r\n\r\nUPDATE:\r\nAlso additional crash issues of `nil` IDs in https://github.com/wordpress-mobile/WordPress-iOS/pull/4443\r\n\r\n@alexcurylo please review the simple `nil` checks added here for protection against crashing.\r\n\r\n@daniloercoli Please take a look at these changes and refactor for what should be tracked or passed when the `dotComID` returns `nil` instead."},"comment":{"url":"https://api.github.com/repos/wordpress-mobile/WordPress-iOS/issues/comments/161233822","html_url":"https://github.com/wordpress-mobile/WordPress-iOS/pull/4537#issuecomment-161233822","issue_url":"https://api.github.com/repos/wordpress-mobile/WordPress-iOS/issues/4537","id":161233822,"user":{"login":"daniloercoli","id":518232,"avatar_url":"https://avatars.githubusercontent.com/u/518232?v=3","gravatar_id":"","url":"https://api.github.com/users/daniloercoli","html_url":"https://github.com/daniloercoli","followers_url":"https://api.github.com/users/daniloercoli/followers","following_url":"https://api.github.com/users/daniloercoli/following{/other_user}","gists_url":"https://api.github.com/users/daniloercoli/gists{/gist_id}","starred_url":"https://api.github.com/users/daniloercoli/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daniloercoli/subscriptions","organizations_url":"https://api.github.com/users/daniloercoli/orgs","repos_url":"https://api.github.com/users/daniloercoli/repos","events_url":"https://api.github.com/users/daniloercoli/events{/privacy}","received_events_url":"https://api.github.com/users/daniloercoli/received_events","type":"User","site_admin":false},"created_at":"2015-12-02T09:28:24Z","updated_at":"2015-12-02T09:28:24Z","body":"Sorry about these crashes :(  Weird that both me and Aaron didn't get those while reviewing the changes.\r\n\r\n"}}