Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork3.2k
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
-
This is a big no-no in our database design of models: In model Can anybody explain why this happened? |
Normalize created_by and changed_by in model PageContent? Yes 100% No (please explain why you're against) 0% 3 votes· |
BetaWas this translation helpful?Give feedback.
All reactions
Replies: 2 comments 5 replies
-
I would suspect this is to handle users being deleted so that the name of the person persists. |
BetaWas this translation helpful?Give feedback.
All reactions
-
If someone needed to keep information on former users would the Django way not be to deactivate the user? |
BetaWas this translation helpful?Give feedback.
All reactions
-
Yes historically that's how I've handled things. |
BetaWas this translation helpful?Give feedback.
All reactions
-
I agree with@fsbraun that users shall be deactivated. If they want to be removed from the database (GDPR compliance), we might add a function to hash their name, email, etc. on some way. But users shall not be removed. I would even argue to use |
BetaWas this translation helpful?Give feedback.
All reactions
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
-
That's an interesting take. I can imagine making this configurable and being reused throughout the CMS (esp. within versioning). Options would be versioning has a similar setting to prevent deleting historical versions. |
BetaWas this translation helpful?Give feedback.
All reactions
-
|
BetaWas this translation helpful?Give feedback.
All reactions
-
I'd be ok with not allowing to remove users as a structure but then you would definitely need to roll out an action to scramble the personal data in the same release. The only reason I see to store the users' names instead of references would be to have a better fidelity of the audit trail. A user changing their name to obscure something would still be recorded as it was at the time. It's pretty contrived I must agree. |
BetaWas this translation helpful?Give feedback.