search

Cheneth / coup-online

An online port of the card game 'Coup'
MIT License
99 stars 40 forks source link

Fix bug where Contessa is always allocated to 1st player #39

Closed sjwarner closed 1 year ago

sjwarner commented 2 years ago

This fixes #37.

In playing with the app, I found the same issue as described in #37. Essentially, it seems like the first user's first card will always be the Contessa. I think this is a bug in the shuffle loop.

I've swapped this to use a Schwartzian transform, which is quite functional. For the small size of the deck, this should be fine.

THM0224 commented 1 year ago

Please merge this! We've experienced this every time as well.