Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-41710

Assignments: Cancelling peer review with an attachment behavior needs to be modified

    XMLWordPrintable

    Details

    • Test Plan:
      Hide
      1. Create a peer review assignment
      2. Have several students submit the assignment
      3. As one of the students who submitted:
        • Click to do the peer review
        • Add a grade
        • Add some text
        • Attach a file
        • Click Cancel Changes
      4. Depending on how this JIRA is fixed, either see a warning message or have the items cleared

       

      Show
      Create a peer review assignment Have several students submit the assignment As one of the students who submitted: Click to do the peer review Add a grade Add some text Attach a file Click Cancel Changes Depending on how this JIRA is fixed, either see a warning message or have the items cleared  

      Description

      With the changes made to the "Cancel Changes" button in SAK-41640, if the student attached a file to a peer review and cancels changes, all information is retained and instead of the status being "Not started", it is "Draft - in progress". 

       

      Attaching a file seems to act like a save". If the attachment cannot be removed, then the student needs to take multiple steps to clear the peer review and a warning should be displayed: 

      "You must remove your attached file before canceling in order to reset this review."

      Currently, this is what the student needs to do in order to set the peer review back to the "Not started" state:

      1. Re-enter the peer review assignment
      2. Remove the attached file
      3. After the file removal refresh, remove any text added
      4. Remove the grade assigned
      5. Save

      #4 does not currently work 

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  Unassigned Unassigned
                  Reporter:
                  agschmidt Andrea Schmidt
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                    Dates

                    Created:
                    Updated:

                      Git Integration