fix: Add exam_name and ready_to_resume to Inst Dash Special Exams tab#38505
fix: Add exam_name and ready_to_resume to Inst Dash Special Exams tab#38505brianjbuck-wgu wants to merge 2 commits intoopenedx:masterfrom
Conversation
|
Thanks for the pull request, @brianjbuck-wgu! This repository is currently maintained by Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review. 🔘 Get product approvalIf you haven't already, check this list to see if your contribution needs to go through the product review process.
🔘 Provide contextTo help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:
🔘 Get a green buildIf one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green. DetailsWhere can I find more information?If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources: When can I expect my changes to be merged?Our goal is to get community contributions seen and reviewed as efficiently as possible. However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:
💡 As a result it may take up to several weeks or months to complete a review and merge your PR. |
There was a problem hiding this comment.
Pull request overview
Adds additional attempt metadata to the Instructor Dashboard “Special Exams” API responses to support UI needs (issue #38504), by exposing exam display name and whether an attempt can be resumed.
Changes:
- Expose
exam_nameonExamAttemptSerializer(fromproctored_exam.exam_name). - Expose
ready_to_resumeonExamAttemptSerializerand add basic response assertions in existing API tests. - Update the OpenAPI spec to document the new response fields.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
lms/djangoapps/instructor/views/serializers_v2.py |
Adds exam_name and ready_to_resume fields to the attempts serializer used by both attempts endpoints. |
lms/djangoapps/instructor/tests/views/test_special_exams_api_v2.py |
Extends API tests to assert the new response fields are present. |
lms/djangoapps/instructor/docs/references/instructor-v2-special-exams-api-spec.yaml |
Documents the new ExamAttempt response properties. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
054e7ae to
16ec12e
Compare
Description
Adds the fields
exam_nameandready_to_resumeto:Closes: #38504