Skip to content

Commit

Permalink
build: Assume that SoftwareSecure PII is retained (to fix annotations)
Browse files Browse the repository at this point in the history
Part of: openedx#1241
  • Loading branch information
kdmccormick committed Nov 12, 2024
1 parent 0b89b87 commit 16e0321
Showing 1 changed file with 15 additions and 9 deletions.
24 changes: 15 additions & 9 deletions edx_proctoring/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -685,10 +685,12 @@ class ProctoredExamSoftwareSecureReview(TimeStampedModel):
This is where we store the proctored exam review feedback
from the exam reviewers
.. pii: reviews contain video of the exam
retirement to be implemented in https://openedx.atlassian.net/browse/EDUCATOR-4776
.. pii: Reviews contain video of the exam.
Retirement was supposed to be implemented in https://openedx.atlassian.net/browse/EDUCATOR-4776,
but it is not clear that is was. We pessimistically assume that it was NOT retired, but this
assumption should be revisited in https://github.com/openedx/edx-proctoring/issues/1241.
.. pii_types: video
.. pii_retirement: to_be_implemented
.. pii_retirement: retained
"""

# which student attempt is this feedback for?
Expand Down Expand Up @@ -758,10 +760,12 @@ class ProctoredExamSoftwareSecureReviewHistory(TimeStampedModel):
"""
When records get updated, we will archive them here
.. pii: reviews contain video of the exam
retirement to be implemented in https://openedx.atlassian.net/browse/EDUCATOR-4776
.. pii: Reviews contain video of the exam.
Retirement was supposed to be implemented in https://openedx.atlassian.net/browse/EDUCATOR-4776,
but it is not clear that is was. We pessimistically assume that it was NOT retired, but this
assumption should be revisited in https://github.com/openedx/edx-proctoring/issues/1241.
.. pii_types: video
.. pii_retirement: to_be_implemented
.. pii_retirement: retained
"""

# which student attempt is this feedback for?
Expand Down Expand Up @@ -804,10 +808,12 @@ class ProctoredExamSoftwareSecureComment(TimeStampedModel):
This is where we store the proctored exam review comments
from the exam reviewers
.. pii: comment contains free-form text which could be personally-identifying
retirement to be implemented in https://openedx.atlassian.net/browse/EDUCATOR-4776
.. pii: Comment contains free-form text which could be personally-identifying.
Retirement was supposed to be implemented in https://openedx.atlassian.net/browse/EDUCATOR-4776,
but it is not clear that is was. We pessimistically assume that it was NOT retired, but this
assumption should be revisited in https://github.com/openedx/edx-proctoring/issues/1241.
.. pii_types: other
.. pii_retirement: to_be_implemented
.. pii_retirement: retained
"""

# which student attempt is this feedback for?
Expand Down

0 comments on commit 16e0321

Please sign in to comment.