search

homeroom-live / homeroom

Live streaming for education.
0 stars 0 forks source link

Add relational dependencies `onDelete` in datamodel. #21

Closed maticzav closed 6 years ago

maticzav commented 6 years ago
maticzav commented 6 years ago

@hoodsy should we delete classrooms if teachers decide to leave Homeroom?

maticzav commented 6 years ago

What about messages? I would say it depends on functionality that we want to have. Not sure how GDPR covers this though...

hoodsy commented 6 years ago

I'm always hesitant with deletes – I'd rather archive everything. No clue about GDPR

That way if they come back, they don't skip a beat

maticzav commented 6 years ago

Agree! Maybe we should add archieved field to every node or something similar and filter things out internally; I think we could still CASCADE deletes on Prisma in case we decide to delete some objects manually not to trash our database.

maticzav commented 6 years ago

Conclusion

We will instead enforce internal deletes as a field property. We should still cascade deletes on Prisma, though, as to keep the database clean in an occurrence of manual data deletion.