mineflayer
mineflayer copied to clipboard
Sign example and implementation fix
There are some things that did not work with the sign example and the sign implementation.
The example was broken and did not work for all versions. It also had references to painting blocks (?) that do not seam to exist anymore.
The updateSign function had an issue where it would stringify the sign text in versions 1.9 and above. But sign text is send as a string after 1.8 so empty lines showed up as ""
on a sign. updateSign also had an issue where it would brick node-minecraft-protocol by trying to send sign text lines that where undefined.
Someone could also remake the sign tests. But I don't have a lot of time right now.
can you please rebase on master ?
So this removes painting completely from all examples I think we should have an example with paintings, maybe another one than this one
todo: Make example simpler
Should I make updateSign and place sign take in an array off strings for the sign text lines? I feel like this would a lot more sense then having to put newline characters into the sign.
For what it's worth, as an end user of the library, an array of strings seems more user-friendly and less brittle.
This should be done. Waiting for someone to merge or suggest changes.
can you rebase and push to retrigger ci ?
Did you delete the commits ? What happened to the bot.placeSign function?
closed with label "was good"
Maybe someone will pick it up someday