{"action":"created","issue":{"url":"https://api.github.com/repos/edx/edx-proctoring/issues/309","repository_url":"https://api.github.com/repos/edx/edx-proctoring","labels_url":"https://api.github.com/repos/edx/edx-proctoring/issues/309/labels{/name}","comments_url":"https://api.github.com/repos/edx/edx-proctoring/issues/309/comments","events_url":"https://api.github.com/repos/edx/edx-proctoring/issues/309/events","html_url":"https://github.com/edx/edx-proctoring/pull/309","id":178764799,"number":309,"title":"Introduce id verification step for proctored exams","user":{"login":"andy-armstrong","id":5985072,"avatar_url":"https://avatars.githubusercontent.com/u/5985072?v=3","gravatar_id":"","url":"https://api.github.com/users/andy-armstrong","html_url":"https://github.com/andy-armstrong","followers_url":"https://api.github.com/users/andy-armstrong/followers","following_url":"https://api.github.com/users/andy-armstrong/following{/other_user}","gists_url":"https://api.github.com/users/andy-armstrong/gists{/gist_id}","starred_url":"https://api.github.com/users/andy-armstrong/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andy-armstrong/subscriptions","organizations_url":"https://api.github.com/users/andy-armstrong/orgs","repos_url":"https://api.github.com/users/andy-armstrong/repos","events_url":"https://api.github.com/users/andy-armstrong/events{/privacy}","received_events_url":"https://api.github.com/users/andy-armstrong/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":17,"created_at":"2016-09-23T01:52:12Z","updated_at":"2016-09-29T23:17:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/edx/edx-proctoring/pulls/309","html_url":"https://github.com/edx/edx-proctoring/pull/309","diff_url":"https://github.com/edx/edx-proctoring/pull/309.diff","patch_url":"https://github.com/edx/edx-proctoring/pull/309.patch"},"body":"## [TNL-5083](https://openedx.atlassian.net/browse/TNL-5083)\r\n\r\n### Description\r\n\r\nThis change introduces a new id verification step when rendering a proctored exam that is shown when the user has not yet passed verification. It also updates the subsequent instruction step to remove the unnecessary recommendation to ensure that the learner is verified.\r\n\r\n### Sandbox\r\n- [x] https://andy-armstrong.sandbox.edx.org/courses/course-v1:AndyA+AAPT+1/courseware/42427a6478c14bc8918c8f1389d34435/9d5f4d5cdeb94d03b2c28a65620dd1f1/\r\n\r\nYou can use the four test users which are already in the various states:\r\n```\r\nverified_id_none@example.com\r\nverified_id_failed@example.com\r\nverified_id_pending@example.com\r\nverified@example.com\r\n```\r\n\r\n### Testing\r\n~~- [x] i18n~~\r\n- [x] Unit, integration, acceptance tests as appropriate\r\n\r\n### Reviewers\r\nIf you've been tagged for review, please check your corresponding box once you've given the :+1:.\r\n- [x] Code review: @dianakhuang \r\n- [x] Code review: @robrap \r\n- [x] Doc Review: @catong \r\n- [ ] UX review: @chris-mike \r\n- [ ] Product review: @marcotuts \r\n\r\nFYI: @douglashall \r\n\r\n### Post-review\r\n- [x] Rebase and squash commits"},"comment":{"url":"https://api.github.com/repos/edx/edx-proctoring/issues/comments/250619024","html_url":"https://github.com/edx/edx-proctoring/pull/309#issuecomment-250619024","issue_url":"https://api.github.com/repos/edx/edx-proctoring/issues/309","id":250619024,"user":{"login":"andy-armstrong","id":5985072,"avatar_url":"https://avatars.githubusercontent.com/u/5985072?v=3","gravatar_id":"","url":"https://api.github.com/users/andy-armstrong","html_url":"https://github.com/andy-armstrong","followers_url":"https://api.github.com/users/andy-armstrong/followers","following_url":"https://api.github.com/users/andy-armstrong/following{/other_user}","gists_url":"https://api.github.com/users/andy-armstrong/gists{/gist_id}","starred_url":"https://api.github.com/users/andy-armstrong/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andy-armstrong/subscriptions","organizations_url":"https://api.github.com/users/andy-armstrong/orgs","repos_url":"https://api.github.com/users/andy-armstrong/repos","events_url":"https://api.github.com/users/andy-armstrong/events{/privacy}","received_events_url":"https://api.github.com/users/andy-armstrong/received_events","type":"User","site_admin":false},"created_at":"2016-09-29T23:17:22Z","updated_at":"2016-09-29T23:17:22Z","body":"@marcotuts discovered that I'd put the id verification step in the wrong place in the workflow. It was showing before the user chose to enter a proctored exam, rather than being shown immediately after they make that choice. I've fixed the code and the tests, and I've updated the sandbox. Great catch, @marcotuts!\r\n\r\nFYI @dianakhuang @robrap @catong @chris-mike."}}
{"actor":{"display_login":"andy-armstrong"}}