{"action":"created","issue":{"url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues/625","labels_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues/625/labels{/name}","comments_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues/625/comments","events_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues/625/events","html_url":"https://github.com/Cal-CS-61A-Staff/ok/pull/625","id":117541823,"number":625,"title":"Auto Promotion of Backups","user":{"login":"Sumukh","id":882381,"avatar_url":"https://avatars.githubusercontent.com/u/882381?v=3","gravatar_id":"","url":"https://api.github.com/users/Sumukh","html_url":"https://github.com/Sumukh","followers_url":"https://api.github.com/users/Sumukh/followers","following_url":"https://api.github.com/users/Sumukh/following{/other_user}","gists_url":"https://api.github.com/users/Sumukh/gists{/gist_id}","starred_url":"https://api.github.com/users/Sumukh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Sumukh/subscriptions","organizations_url":"https://api.github.com/users/Sumukh/orgs","repos_url":"https://api.github.com/users/Sumukh/repos","events_url":"https://api.github.com/users/Sumukh/events{/privacy}","received_events_url":"https://api.github.com/users/Sumukh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-11-18T08:58:45Z","updated_at":"2015-11-19T04:56:39Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls/625","html_url":"https://github.com/Cal-CS-61A-Staff/ok/pull/625","diff_url":"https://github.com/Cal-CS-61A-Staff/ok/pull/625.diff","patch_url":"https://github.com/Cal-CS-61A-Staff/ok/pull/625.patch"},"body":"This fix has been deployed onto staging and seems to work. \r\n\r\nTODO:\r\n - Promote more than just the last backup. Attempt to hunt for the backup closest to the due-date. \r\n - Lower Priority: Also consider submissions first if any exist (to handle edge cases where lock_dates/due_dates where off) \r\n- Write tests for auto promotion (coveralls fell below 89%) \r\n"},"comment":{"url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues/comments/157951023","html_url":"https://github.com/Cal-CS-61A-Staff/ok/pull/625#issuecomment-157951023","issue_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues/625","id":157951023,"user":{"login":"Sumukh","id":882381,"avatar_url":"https://avatars.githubusercontent.com/u/882381?v=3","gravatar_id":"","url":"https://api.github.com/users/Sumukh","html_url":"https://github.com/Sumukh","followers_url":"https://api.github.com/users/Sumukh/followers","following_url":"https://api.github.com/users/Sumukh/following{/other_user}","gists_url":"https://api.github.com/users/Sumukh/gists{/gist_id}","starred_url":"https://api.github.com/users/Sumukh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Sumukh/subscriptions","organizations_url":"https://api.github.com/users/Sumukh/orgs","repos_url":"https://api.github.com/users/Sumukh/repos","events_url":"https://api.github.com/users/Sumukh/events{/privacy}","received_events_url":"https://api.github.com/users/Sumukh/received_events","type":"User","site_admin":false},"created_at":"2015-11-19T04:56:39Z","updated_at":"2015-11-19T04:56:39Z","body":"@knrafto - I see - I've removed the try/catch (we needed it before when we just got an arbitrary backup - but no longer) \r\n\r\nAside from tests - is there anything you'd like to see made in this PR before merging? "}}