Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Players shouldn't get fully deleted with delete command #346

Open
Kowlin opened this issue Aug 21, 2024 · 1 comment
Open

Players shouldn't get fully deleted with delete command #346

Kowlin opened this issue Aug 21, 2024 · 1 comment
Labels
core:bot core:models package:admin All /admin commands and its configuration options package:players Package with /player commands priority:high

Comments

@Kowlin
Copy link
Contributor

Kowlin commented Aug 21, 2024

For the sake of keeping a track record of players and combat cheating, we should not fully delete the player from the database and keep their Player and Discord IDs on file. To cross reference with other data in our database, such as trade history.

As a side-note for the argument of GDPR; Discord IDs aren't personal identification and user's identities cannot be traced back to them via a Discord ID alone. The same applies to our player IDs which are only strongly linked to Discord IDs. We also cannot remove Discord IDs from our database because of anti-cheat purposes.

@Kowlin Kowlin added package:players Package with /player commands core:models core:bot package:admin All /admin commands and its configuration options priority:high labels Aug 21, 2024
@laggron42
Copy link
Member

The information contained under the player table is very light, only IDs and a few settings. What should be the behavior of a soft deletion then, resetting inventory? Also, should a player decide to come back after deletion, should he be given a new ID or the old one reused? 🤔

@laggron42 laggron42 linked a pull request Nov 28, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core:bot core:models package:admin All /admin commands and its configuration options package:players Package with /player commands priority:high
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants