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

Update state without checking values #62

Merged
merged 1 commit into from
Jul 7, 2021

Conversation

timotheeg
Copy link
Contributor

When starting capture mid-game, gameid stays at 0, and game state incorrectly stays at MENU.

Opening a small changeset for discussion.

@alex-ong
Copy link
Owner

alex-ong commented Aug 2, 2020

I believe we discussed setting default state to in game, and keeping level != Null + lines == 000 or 25 and score == 000000 to start game? Could you do that (or I can too...)

@timotheeg
Copy link
Contributor Author

Oh I forgot about this PR 😅 Let me update it to what we discussed.

@timotheeg timotheeg force-pushed the simpler_state_transition branch 3 times, most recently from 1bab416 to 0b1100a Compare August 2, 2020 22:55
@timotheeg
Copy link
Contributor Author

Done

@timotheeg
Copy link
Contributor Author

@alex-ong this is ready to merge I think 😃 Pls check

@alex-ong alex-ong merged commit 488beeb into alex-ong:master Jul 7, 2021
@alex-ong
Copy link
Owner

alex-ong commented Jul 7, 2021

I didnt check, i'm going to blind merge 🍷

@timotheeg
Copy link
Contributor Author

Lol, I completely forgot this PR! I hope it doesn't introduce a bug 😅

@timotheeg timotheeg deleted the simpler_state_transition branch July 12, 2021 08:14
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.

2 participants