instructure / canvas-lms

The open LMS by Instructure, Inc.
https://github.com/instructure/canvas-lms/wiki
GNU Affero General Public License v3.0
5.68k stars 2.5k forks source link

PUT to update assignment submission rubric points failing silently #1434

Open hcientist opened 5 years ago

hcientist commented 5 years ago

Summary:

I have previously submitted PUT requests to the https://canvas.instructure.com/doc/api/submissions.html#method.submissions_api.update and had great success updating students' rubric points. Now, this same API call is failing silently (it responds with HTTP 200, even though the entries are not persisted).

Steps to reproduce:

  1. make a PUT as documented in https://canvas.instructure.com/doc/api/submissions.html#method.submissions_api.update but specifically with the info like rubric_assessment[crit1][points]=3 set.
  2. get back HTTP 200
  3. check the grade or the rubric values in speedgrader and see no information added/updated.

Expected behavior:

Either get an error code due to a problem with the response, or when getting success code, update the values.

Actual behavior:

Get success code but no data persisted.

Additional notes:

This may be related to https://github.com/instructure/canvas-lms/issues/1304 ? I am trying to do this with https://github.com/JMU-CS/canvas-rubric-filler which worked nicely for us in December (and i'm just trying it again for the first time). Interesting note: I have a test course in which I previously had success using this script to post rubric points and that same test course and assignment are now demonstrating the actual (unexpected) behavior.

stale[bot] commented 2 years ago

Thanks for contributing to this issue. As it has been 2 years since the last activity, we are automatically closing the issue in 30 days. This is often because the request was already solved in some way and it just wasn't updated or it's no longer applicable. If that's not the case, please respond before the issue is closed, or post a message on the mailing list. We'll gladly take a look again!