Skip to content

Conversation

@vmcj
Copy link
Member

@vmcj vmcj commented Nov 30, 2025

In all years that I've worked with DOMjudge I've never seen a good reason to collect the email separately. So I propose to remove the collection in the next major release.

We keep storing the emails in development dumps and although it doesn´t hurt that much it is a bit more privacy sensitive than most other fields IMHO.

@vmcj vmcj force-pushed the privacy_online_register branch from 8cdf600 to 0ce95f8 Compare November 30, 2025 12:12
@vmcj vmcj force-pushed the privacy_online_register branch from a7e73a1 to a121a23 Compare December 21, 2025 12:14
@Kevinjil
Copy link
Contributor

This is one of those features I have no idea if anyone in the world is using it. I can imagine use cases when self-signup enabled and, e.g. after contest, you'd like to sent out mail to whoever entered an email address.

@vmcj vmcj force-pushed the privacy_online_register branch from 716298e to b401811 Compare December 21, 2025 15:29
@vmcj vmcj force-pushed the privacy_online_register branch from b401811 to b7f5a11 Compare December 21, 2025 15:39
@meisterT
Copy link
Member

Is entering an email address mandatory right now? If so, could me make it optional instead of dropping it?

There might be installations out there which suddenly lose data that may be important to them (because that's how they can communicate with their users).

@vmcj
Copy link
Member Author

vmcj commented Dec 22, 2025

Is entering an email address mandatory right now? If so, could me make it optional instead of dropping it?

It's already optional, we still get them in our dumps as it's implied with such installation that you might get an confirmation email?

There might be installations out there which suddenly lose data that may be important to them (because that's how they can communicate with their users).

In the 2nd commit we don't drop the column if it exists, only on new installs the column is not created. That is the intent at least.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants