ramou / ChessMod

Adding Pippin Barr's Chesses to minecraft.
MIT License
3 stars 3 forks source link

Add functionality to broadcast messages to nearby players when issuing a notify-client of change #175

Closed ramou closed 1 month ago

ramou commented 2 months ago

There's a lot of code in ChessBlock for announcing stuff to the player who does thing, but there's also the matter of the player making moves, now particularly with the quantum chessboard where that'll be reflected potentially quite far away. Thinking about the responsibility separation, it seems to make more sense to allow the entity to take care of that, as it knows where it is and how to find nearby players properly, and that logic seems mostly tied to its chessboardness vs. it being a thing in the world... the events in block just conveniently had the triggering player to spam a message at.

Anyway, the basic idea was from @riri404 and thinking on it, I think this'd be a nice and convenient way to do it. We might even write a thing that translates the specific board move to proper chess-speak, but I'll leave that to another issue if someone's motivated to make that a priority.