ctm / mb2-doc

Mb2, poker software
https://devctm.com
7 stars 2 forks source link

Announces game as "Dealer's Choice" #1359

Closed ctm closed 6 months ago

ctm commented 6 months ago

Fix so it Dealer's Choice doesn't announce "Dealer's Choice" as the game that has been chosen.

This is a regression that I introduced when I added the ability for people to choose the game in advance. It is caused by me adding the ChosenGame message for Dealer's Choice. Mb2 used to send that only for Paradise Pick'em and Wonky Donkey. I added it so that the client could know when a choice had been consumed so it could resend it, but I couldn't get that to work due to the action synchronizer. Turns out, the server doesn't need to consume the choice anyway. D'oh!

ctm commented 6 months ago

Fixed in master. It'll go out in this evening's deploy.