Skip to content
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

Randomly choose color when player who played +4 or wildcard get skipped #64

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

eaocto
Copy link

@eaocto eaocto commented Nov 28, 2018

Before, if this condition happens then players would have to keep drawing until someone get the same card that was played by the skipped player.

… skipped player who played +4 or wildcard get skipped before choosing color.
@DO97
Copy link

DO97 commented Dec 3, 2018

Is it possible to show the randomly choosed color?

Bug - In this case the next player will not draw four cards.

@jh0ker
Copy link
Owner

jh0ker commented Dec 5, 2018

Hey, thanks for your contribution! It would be cool to have this also for players that leave the game

…lor" step, +4 will be correctly given to the next player.

Game will also randomly choose color if player leave without choosing color after playing wildcard or +4.
@eaocto
Copy link
Author

eaocto commented Dec 13, 2018

@DO97 i have added randomed color message. Also now the next player will properly get +4

@jh0ker i've extended this to leaving player

There's a known bug; if +4 was played and the player left and the next player call bluff, it will be targeted to the player before the leaving player.

@DO97
Copy link

DO97 commented Dec 16, 2018

Thank you!

@eaocto Does it work when a player get automatically skipped in Sanic mode (I mean the message of the random color)?

Is it possible to also add this function in case a player is kicked from the game?

Do you have a Telegram username?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants