-
Notifications
You must be signed in to change notification settings - Fork 16
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
Fix #26 & should fix #31 #32
Conversation
Thanks for the PR, however this solution will not be sufficient, the best alternative would be to reload the session data when the player re-logs in. Your solution is to remove the session when the player exits the server, which will partially solve the problem, but if you load the data when the player is offline (for example using the |
I fix it when I'm home, the function to reload session is Session->loadUserData() right? |
Btw great job with this plugin 👍😁 |
Removed quit event
Added contains function to check if session is already in there
made loadUserData public for EventListener access them
@IvanCraft623 is this acceptable? |
never mind this dosn't work |
Changes: