TICKET-583: Include due date in LTI gradebook sync#7872
TICKET-583: Include due date in LTI gradebook sync#7872
Conversation
Coverage Report for CI Build 24734649524Coverage increased (+0.001%) to 91.714%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsNo coverage regressions found. Coverage Stats💛 - Coveralls |
5ac1ae4 to
100ca98
Compare
78c0eda to
ca824e6
Compare
donny-wong
left a comment
There was a problem hiding this comment.
Thanks for this. However, when I tested it locally, the due date was still not syncing to Canvas. Are you able to sync on your end?
ca824e6 to
c29ef77
Compare
c29ef77 to
33fe566
Compare
Hi Donny, I have updated PR and posted a screenshot. |
33fe566 to
41cbd44
Compare
Thanks @Naragod , strange, I am still unable to sync it. Maybe you can show me tomorrow.
Hi @Naragod , strange, I still cannot sync it on my end. Would you have some idea why that would be? |
Proposed Changes 🐛 Bug fix
When creating or updating a gradebook item via LTI, MarkUs sends a line item payload to Canvas containing the assignment label, resource ID, and maximum score. The
dueAtfield supported by the LTI Assignment and Grade Services (AGS) specification was missing from this payload, and the due date set in MarkUs remained absent on the corresponding Canvas assignment. The fix addsdueAtas an ISO 8601 timestamp to the payload when the assessment has a due date, ensuring it is synced during both line item creation and updates.Screenshots of your changes (if applicable)
Associated documentation repository pull request (if applicable)
Type of Change
(Write an
Xor a brief description next to the type or types that best describe your changes.)Checklist
(Complete each of the following items for your pull request. Indicate that you have completed an item by changing the
[ ]into a[x]in the raw text, or by clicking on the checkbox in the rendered description on GitHub.)Before opening your pull request:
After opening your pull request:
Questions and Comments
(Include any questions or comments you have regarding your changes.)