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

Stats store info with nickname instead of user id #2

Open
asteriskman7 opened this issue Mar 4, 2021 · 2 comments
Open

Stats store info with nickname instead of user id #2

asteriskman7 opened this issue Mar 4, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@asteriskman7
Copy link
Member

Currently, stats are saved vs nickname. This breaks when users change their nickname.

This needs to change to saving stats via user id. When stats are displayed convert the user id to nickname. May need additional code to deal with user ids that are no longer in the server.

Should include code to do a 1 time conversion of old stats to new format.

@Ishadijcks
Copy link
Collaborator

Maybe whenever a user with such an existing name sends a message, it moves the stat to that id and deletes the name record?

Or a one-off command that checks everyone on the server (if that's possible)

@asteriskman7 asteriskman7 added the bug Something isn't working label Mar 4, 2021
@asteriskman7
Copy link
Member Author

Yeah, whatever reasonable logic to get from where we are to where we should be is ok. This is just for fun so it doesn't need to be perfect.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants