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

Fix Museum Mode Fast Tap Issue (appears to be specific to drawing workflow) #311

Merged
merged 1 commit into from
Dec 4, 2019

Conversation

chelseatroy
Copy link
Contributor

@chelseatroy chelseatroy commented Dec 3, 2019

Children can mash the submit button all they want, but it disables after the first press until a new image loads

  • I also checked whether other workflows have this problem. I was not able to reproduce the issue in the other workflows.

Fixes #310.

Invision Mock-ups:

Describe your changes.

Review Checklist

  • Does it work in Android and iOS?
  • Can you rm -rf node_modules/ && npm install and app works as expected?
  • Are tests passing?

…ter the first press until a new image loads

+ I also checked whether other workflows have this problem. I was not able to reproduce the issue in the other workflows.
@chelseatroy chelseatroy force-pushed the fix-museum-mode-fast-tap-submit branch from 7152298 to 7e74b3d Compare December 4, 2019 16:15
@wgranger
Copy link
Contributor

wgranger commented Dec 4, 2019

LGTM. Nice and succinct fix. I've posted a screenshot of the error message from the simulator in the original issue.

@wgranger wgranger merged commit 0dcbcc2 into master Dec 4, 2019
@wgranger wgranger deleted the fix-museum-mode-fast-tap-submit branch December 4, 2019 17:35
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.

Crashing Bug: Double Submit
2 participants