-
Notifications
You must be signed in to change notification settings - Fork 38
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
LCD appears broken #55
Comments
I can't find any problems at testing with a newly placed lcd. |
Old LCD displays no longer worked. All I did was hook up a chest to a LCD screen to quickly see if taking out/storing items produces the expected text. |
Ah, I see.
This means that old constructions won't work anymore. (I'm sorry that I haven't spent again time to review that PR btw. >_<) |
Ah, yes, that was my change, wasn’t it? Yes, it does unfortunately break existing builds, but I submitted it thinking that the improvement was worthwhile (IMO the old string-based message format simply didn’t give enough information to do some of the useful things with Digilines chests that people might want, and expanding the string-based format would break existing parsers anyway, so why not just switch to tables while we’re at it?). I didn’t realize people would directly connect a Digilines chest to an LCD without an intervening Luacontroller, and it’s true that doesn’t work any more. As far as informing players, does Digilines have a news page, changelog, or something? I’m not a member of the Minetest forum; maybe that’s where the info is supposed to go? |
After all the merges, it appears the LCD no longer functions properly, and I don't understand why.
@numberZero @DS-Minetest
The text was updated successfully, but these errors were encountered: