Tinson Lai
Tinson Lai
I've added two new CMake policies ([CMP0077](https://cmake.org/cmake/help/v3.13/policy/CMP0077.html) and [CMP0126](https://cmake.org/cmake/help/v3.21/policy/CMP0126.html)) for better integration as a subproject after PR #85. --- I've committed an unwanted change in the previous PR (#85), apologies...
A similar fix proposed previously by #5602 failed the CI tests. I made some amendments, and it is compilable in my environment. It should now correct the problem raised in...
# Improving `canvasapi.submission.Submission` Actually, the [`Submission`](https://canvas.instructure.com/doc/api/submissions.html#Submission) object returned by the Canvas API will potentially [contain an `attachment` attribute](https://community.canvaslms.com/t5/Canvas-Developers-Group/Downloading-Submission-File-Attachment/td-p/117413), which is, in fact, a list of [`File`](https://canvas.instructure.com/doc/api/files.html#File) objects. Currently, the created...
# Proposed Changes This PR addresses issue #511 by adding constructor to `canvasapi.submission.Submission`, and process the `attachments` field retrieved from the JSON object. It will convert each plain `dict` to...
# Proposed Changes This PR addresses issue #490 by adding two additional methods to `canvasapi.submission.Submission`. To complete the testing, I've also added the optional "submission_comments" field to the original submission...
**What resource needs additional coverage?** Submission Comment **What endpoints need to be covered?** `PUT /api/v1/courses/:course_id/assignments/:assignment_id/submissions/:user_id/comments/:id` `DELETE /api/v1/courses/:course_id/assignments/:assignment_id/submissions/:user_id/comments/:id` They are documented [here](https://canvas.instructure.com/doc/api/submission_comments.html). After searching the repository, I realised that this endpoint...