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

Store players with UUID instead of in-game name #24

Open
BrainStone opened this issue Sep 21, 2018 · 5 comments
Open

Store players with UUID instead of in-game name #24

BrainStone opened this issue Sep 21, 2018 · 5 comments

Comments

@BrainStone
Copy link

Players should never be stored with only the IGN! The name can change. However the UUID cannot. That's why it's important to use a UUID to identify the players.

@Yann151924
Copy link
Member

Thanks for the suggestion, but the goal is to keep compatibility with GiveAnythingListener and i don't think the plugin on spigot has this support. But we can plan to port our version to spigot to add this kind of support

@BrainStone
Copy link
Author

If I'm not mistaken, then this plugin is Sponge only. So why worry about compatibility to a competitor on a different platform?
Especially if that competitor lacks basic functionality like UUID support?

Anyways, compatibility to a different plugin isn't nearly as important as proper combatibility to Minecraft itself. And we're at a point where UUID support is expected. Most plugins don't even bother anymore to explicitly state they have UUID support, that's how basic this has become.

@Yann151924
Copy link
Member

The first goal of this plugin was to make GiveAnythingListner to Sponge because we need it 😉 , that why the config look like GAListener and some other stuff. But like i said we can plan to make a bukkit version of this plugin and extend the plugin.
We can appreciate PR on this kind of stuff to make a release faster on this point.

@Rasgnarok
Copy link

Is it possible to add UUID now please? It also means that on name change, players lose their vote milestones... Would be really appreciated. https://github.com/ObliqueNET/Stratos/issues/113

@Yann151924
Copy link
Member

Yann151924 commented Mar 2, 2019

I didn't work on it on the full rewrite but after i fully tested all case and the version is out. I will work on it.

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

No branches or pull requests

3 participants