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.

Sign up for GitHub

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

Jump to bottom

Prompt spymaster to give a hint #3

Closed
deesejohn opened this issue Jan 23, 2021 · 1 comment · Fixed by #14
Closed

Prompt spymaster to give a hint #3

deesejohn opened this issue Jan 23, 2021 · 1 comment · Fixed by #14
Assignees
@deesejohn

Comments

@deesejohn
Copy link
Owner

The spymaster on the guessing team should be prompted to give a hint when there currently is no clue and it is their team's turn to guess.
@deesejohn
Copy link
Owner Author

@skinney87, I'm thinking about a dialog (https://material-ui.com/components/dialogs/) with a form that takes a word: string and number: number. That would http post to a hint endpoint (still needs to be made). What do you think? Would you want to handle any and/or all of the implementation?

@deesejohn deesejohn reopened this Feb 2, 2021
deesejohn added a commit that referenced this issue Feb 3, 2021
@deesejohn
Added api endpoint for hinting #3
b7fa027
@deesejohn deesejohn self-assigned this Feb 3, 2021
@deesejohn deesejohn linked a pull request Feb 3, 2021 that will close this issue
Feature/hinting #3 #14
Merged
deesejohn added a commit that referenced this issue Feb 4, 2021
@deesejohn
Feature/hinting #3 (#14)
41bd930 
* Updated guess to end game #4

* Added death card logic and fixed next turn on guess

* Added api endpoint for hinting #3

* Prompts spymaster for hint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@deesejohn deesejohn

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feature/hinting #3 deesejohn/distributed-codenames
1 participant
@deesejohn