Add data retention period to event information fields #2296
Conversation
|
Code Climate has analyzed commit 63bfb05 and detected 1 issue on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
se-bastiaan
changed the title
| @@ -40,6 +40,15 @@ class RegistrationInformationField(models.Model): | |||
| _("required"), | |||
| ) | |||
|
|
|||
| delete_after = models.DateField( | |||
There was a problem hiding this comment.
Why don't we delete this after a predetermined time? This way it could technically be saved forever if people enter a value that is years in the future.
There was a problem hiding this comment.
Also possible. The reason I opened the issue myself is because we don't have an exact policy on this, and regarding data minimization I figure that some kinds of data might have different retention times than others and this forces organizers to think about it. We could add a validator of max 3 years or something. But I'm open for discussion on this
There was a problem hiding this comment.
I would personally say that 1/2-ish months after the activity ended would also be fine. So many options. As long as it is not 'unlimited'.
There was a problem hiding this comment.
As a constant? Or as max value?
There was a problem hiding this comment.
No, as you can see (we do anonymize it, we don't delete it)
There was a problem hiding this comment.
Then I think 2 months after the event is over we can delete the registration info
There was a problem hiding this comment.
I think we should keep it this way. Force people to think about what to do with this data.
There was a problem hiding this comment.
Isn't it a bit our job to have thought about this for them?
There was a problem hiding this comment.
No, because we don't know what kind of data this is, so we are not able to make a judgement on what the retention period should be.
DeD1rk
force-pushed
the
master
branch
2 times, most recently
from
41d42a7
to
18edd09
Compare
Closes #1949
Summary
Adds a data retention period to event info fields
How to test