{"action":"created","comment":{"url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls/comments/98143781","pull_request_review_id":18777007,"id":98143781,"diff_hunk":"@@ -120,40 +122,78 @@ def submit_assignment(name):\n     group = Group.lookup(current_user, assign)\n     user_ids = assign.active_user_ids(current_user.id)\n     fs = assign.final_submission(user_ids)\n-\n     if not assign.uploads_enabled:\n         flash(\"This assignment cannot be submitted online\", 'warning')\n         return redirect(url_for('.assignment', name=assign.name))\n     if not assign.active:\n         flash(\"It's too late to submit this assignment\", 'warning')\n         return redirect(url_for('.assignment', name=assign.name))\n \n-    form = UploadSubmissionForm()","path":"server/controllers/student.py","position":33,"original_position":33,"commit_id":"243d6a29a11e2505f2e03996fc72fa855f8e2442","original_commit_id":"243d6a29a11e2505f2e03996fc72fa855f8e2442","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},"body":"Should be able to just remove this form entirely once the admin stops using it. ","created_at":"2017-01-27T04:35:31Z","updated_at":"2017-01-27T04:41:27Z","html_url":"https://github.com/Cal-CS-61A-Staff/ok/pull/1046#discussion_r98143781","pull_request_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls/1046","_links":{"self":{"href":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls/comments/98143781"},"html":{"href":"https://github.com/Cal-CS-61A-Staff/ok/pull/1046#discussion_r98143781"},"pull_request":{"href":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls/1046"}}},"pull_request":{"url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls/1046","id":103437187,"html_url":"https://github.com/Cal-CS-61A-Staff/ok/pull/1046","diff_url":"https://github.com/Cal-CS-61A-Staff/ok/pull/1046.diff","patch_url":"https://github.com/Cal-CS-61A-Staff/ok/pull/1046.patch","issue_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues/1046","number":1046,"state":"open","locked":false,"title":"Drag & Drop Uploads + Binary File Upload","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},"body":"Quite a few things left to cleanup. \r\n\r\n184 needs this in production soon. \r\n\r\nThings I want to change: \r\n- Code cleanup - I haven't done a pass over it \r\n- Change the \"Uploading\" animation to use something from pace.js \r\n- @kvchen: Suggested a list of files at the top (useful if you have 3+ files) \r\n- (Maybe not in this PR) Add drag & drop for staff too? \r\n\r\nThings I noticed but don't feel compelled to fix right now\r\n- If you click on the dropzone and get the native file picker - you can't upload folders. \r\n\r\nFuture Ideas:\r\n- Something that looks inside of ZIP files to see the contents. \r\n","created_at":"2017-01-27T04:34:52Z","updated_at":"2017-01-27T04:41:27Z","closed_at":null,"merged_at":null,"merge_commit_sha":"699c67ef9b81a250c245eef0c3924fe0f7714310","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls/1046/commits","review_comments_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls/1046/comments","review_comment_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues/1046/comments","statuses_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/statuses/243d6a29a11e2505f2e03996fc72fa855f8e2442","head":{"label":"Cal-CS-61A-Staff:dropzone","ref":"dropzone","sha":"243d6a29a11e2505f2e03996fc72fa855f8e2442","user":{"login":"Cal-CS-61A-Staff","id":7565272,"avatar_url":"https://avatars.githubusercontent.com/u/7565272?v=3","gravatar_id":"","url":"https://api.github.com/users/Cal-CS-61A-Staff","html_url":"https://github.com/Cal-CS-61A-Staff","followers_url":"https://api.github.com/users/Cal-CS-61A-Staff/followers","following_url":"https://api.github.com/users/Cal-CS-61A-Staff/following{/other_user}","gists_url":"https://api.github.com/users/Cal-CS-61A-Staff/gists{/gist_id}","starred_url":"https://api.github.com/users/Cal-CS-61A-Staff/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Cal-CS-61A-Staff/subscriptions","organizations_url":"https://api.github.com/users/Cal-CS-61A-Staff/orgs","repos_url":"https://api.github.com/users/Cal-CS-61A-Staff/repos","events_url":"https://api.github.com/users/Cal-CS-61A-Staff/events{/privacy}","received_events_url":"https://api.github.com/users/Cal-CS-61A-Staff/received_events","type":"Organization","site_admin":false},"repo":{"id":19992691,"name":"ok","full_name":"Cal-CS-61A-Staff/ok","owner":{"login":"Cal-CS-61A-Staff","id":7565272,"avatar_url":"https://avatars.githubusercontent.com/u/7565272?v=3","gravatar_id":"","url":"https://api.github.com/users/Cal-CS-61A-Staff","html_url":"https://github.com/Cal-CS-61A-Staff","followers_url":"https://api.github.com/users/Cal-CS-61A-Staff/followers","following_url":"https://api.github.com/users/Cal-CS-61A-Staff/following{/other_user}","gists_url":"https://api.github.com/users/Cal-CS-61A-Staff/gists{/gist_id}","starred_url":"https://api.github.com/users/Cal-CS-61A-Staff/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Cal-CS-61A-Staff/subscriptions","organizations_url":"https://api.github.com/users/Cal-CS-61A-Staff/orgs","repos_url":"https://api.github.com/users/Cal-CS-61A-Staff/repos","events_url":"https://api.github.com/users/Cal-CS-61A-Staff/events{/privacy}","received_events_url":"https://api.github.com/users/Cal-CS-61A-Staff/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Cal-CS-61A-Staff/ok","description":"ok.py supports programming projects by running tests, tracking progress, and assisting in debugging.","fork":false,"url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok","forks_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/forks","keys_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/teams","hooks_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/hooks","issue_events_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues/events{/number}","events_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/events","assignees_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/assignees{/user}","branches_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/branches{/branch}","tags_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/tags","blobs_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/git/refs{/sha}","trees_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/statuses/{sha}","languages_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/languages","stargazers_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/stargazers","contributors_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/contributors","subscribers_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/subscribers","subscription_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/subscription","commits_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/commits{/sha}","git_commits_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/git/commits{/sha}","comments_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/comments{/number}","issue_comment_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues/comments{/number}","contents_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/contents/{+path}","compare_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/merges","archive_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/downloads","issues_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues{/number}","pulls_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls{/number}","milestones_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/milestones{/number}","notifications_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/labels{/name}","releases_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/releases{/id}","deployments_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/deployments","created_at":"2014-05-20T18:36:23Z","updated_at":"2017-01-10T01:02:03Z","pushed_at":"2017-01-27T04:34:53Z","git_url":"git://github.com/Cal-CS-61A-Staff/ok.git","ssh_url":"git@github.com:Cal-CS-61A-Staff/ok.git","clone_url":"https://github.com/Cal-CS-61A-Staff/ok.git","svn_url":"https://github.com/Cal-CS-61A-Staff/ok","homepage":null,"size":19894,"stargazers_count":40,"watchers_count":40,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":20,"mirror_url":null,"open_issues_count":73,"forks":20,"open_issues":73,"watchers":40,"default_branch":"master"}},"base":{"label":"Cal-CS-61A-Staff:master","ref":"master","sha":"b7d196d45f9ccab6908f6941c48c4460e33ac388","user":{"login":"Cal-CS-61A-Staff","id":7565272,"avatar_url":"https://avatars.githubusercontent.com/u/7565272?v=3","gravatar_id":"","url":"https://api.github.com/users/Cal-CS-61A-Staff","html_url":"https://github.com/Cal-CS-61A-Staff","followers_url":"https://api.github.com/users/Cal-CS-61A-Staff/followers","following_url":"https://api.github.com/users/Cal-CS-61A-Staff/following{/other_user}","gists_url":"https://api.github.com/users/Cal-CS-61A-Staff/gists{/gist_id}","starred_url":"https://api.github.com/users/Cal-CS-61A-Staff/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Cal-CS-61A-Staff/subscriptions","organizations_url":"https://api.github.com/users/Cal-CS-61A-Staff/orgs","repos_url":"https://api.github.com/users/Cal-CS-61A-Staff/repos","events_url":"https://api.github.com/users/Cal-CS-61A-Staff/events{/privacy}","received_events_url":"https://api.github.com/users/Cal-CS-61A-Staff/received_events","type":"Organization","site_admin":false},"repo":{"id":19992691,"name":"ok","full_name":"Cal-CS-61A-Staff/ok","owner":{"login":"Cal-CS-61A-Staff","id":7565272,"avatar_url":"https://avatars.githubusercontent.com/u/7565272?v=3","gravatar_id":"","url":"https://api.github.com/users/Cal-CS-61A-Staff","html_url":"https://github.com/Cal-CS-61A-Staff","followers_url":"https://api.github.com/users/Cal-CS-61A-Staff/followers","following_url":"https://api.github.com/users/Cal-CS-61A-Staff/following{/other_user}","gists_url":"https://api.github.com/users/Cal-CS-61A-Staff/gists{/gist_id}","starred_url":"https://api.github.com/users/Cal-CS-61A-Staff/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Cal-CS-61A-Staff/subscriptions","organizations_url":"https://api.github.com/users/Cal-CS-61A-Staff/orgs","repos_url":"https://api.github.com/users/Cal-CS-61A-Staff/repos","events_url":"https://api.github.com/users/Cal-CS-61A-Staff/events{/privacy}","received_events_url":"https://api.github.com/users/Cal-CS-61A-Staff/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Cal-CS-61A-Staff/ok","description":"ok.py supports programming projects by running tests, tracking progress, and assisting in debugging.","fork":false,"url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok","forks_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/forks","keys_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/teams","hooks_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/hooks","issue_events_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues/events{/number}","events_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/events","assignees_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/assignees{/user}","branches_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/branches{/branch}","tags_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/tags","blobs_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/git/refs{/sha}","trees_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/statuses/{sha}","languages_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/languages","stargazers_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/stargazers","contributors_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/contributors","subscribers_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/subscribers","subscription_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/subscription","commits_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/commits{/sha}","git_commits_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/git/commits{/sha}","comments_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/comments{/number}","issue_comment_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues/comments{/number}","contents_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/contents/{+path}","compare_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/merges","archive_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/downloads","issues_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues{/number}","pulls_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls{/number}","milestones_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/milestones{/number}","notifications_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/labels{/name}","releases_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/releases{/id}","deployments_url":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/deployments","created_at":"2014-05-20T18:36:23Z","updated_at":"2017-01-10T01:02:03Z","pushed_at":"2017-01-27T04:34:53Z","git_url":"git://github.com/Cal-CS-61A-Staff/ok.git","ssh_url":"git@github.com:Cal-CS-61A-Staff/ok.git","clone_url":"https://github.com/Cal-CS-61A-Staff/ok.git","svn_url":"https://github.com/Cal-CS-61A-Staff/ok","homepage":null,"size":19894,"stargazers_count":40,"watchers_count":40,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":20,"mirror_url":null,"open_issues_count":73,"forks":20,"open_issues":73,"watchers":40,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls/1046"},"html":{"href":"https://github.com/Cal-CS-61A-Staff/ok/pull/1046"},"issue":{"href":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues/1046"},"comments":{"href":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/issues/1046/comments"},"review_comments":{"href":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls/1046/comments"},"review_comment":{"href":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/pulls/1046/commits"},"statuses":{"href":"https://api.github.com/repos/Cal-CS-61A-Staff/ok/statuses/243d6a29a11e2505f2e03996fc72fa855f8e2442"}}}}
{"actor":{"display_login":"Sumukh"}}