search

jdeath / Bedrock-Homeassistant

MIT License
8 stars 5 forks source link

Player names not available #1

Closed energywave closed 3 years ago

energywave commented 3 years ago

First thank you very much! This integration works beautifully. But I'm missing online player names. The core integration doc says they're exposed as attributes in the online players sensor. But with bedrock server there aren't. Can this be implemented?

jdeath commented 3 years ago

Unfortunately not right now. The mcstatus package only provides player names on Java, not on bedrock. If mcstatus (https://github.com/Dinnerbone/mcstatus/) is updated with this capability, then I could update this integration to show them.

Kirbo commented 2 years ago

I think this is supported now, or am I incorrect? https://github.com/Dinnerbone/mcstatus/issues/158 Also noticed that mcstatus supports Bedrock servers now as well, should this make any difference for this project: https://github.com/Dinnerbone/mcstatus/issues/153

jdeath commented 2 years ago

Player names are just for Java, not bedrock. This home assistant integration uses and has always used mcstatus (albeit an older version), just like the built JAVA only one. Since the built-in homeassistant uses an old version, it is a pain to do a pull request to get them to update. I will do that when/if mcstatus is ever updated to handle playernames.