Fix bug with quiz next button skipping questions#2312
Fix bug with quiz next button skipping questions#2312itsyme merged 2 commits intoMarkBind:masterfrom
Conversation
|
Hi @lesterong thanks very much for the work on this PR! |
@yucheng11122017, noted on that! I have edited my comment with the full template. Also, may I clarify what unit tests to add? It seems like the tests for the |
hi @lesterong! the tests were disabled due to some unexpected behaviour with setTimeout being used in the tests causing CI to fail sometimes. perhaps you could test the fix locally but skip it when you commit so that we can unskip them when we solve #2238 |
|
Checked on https://deploy-preview-2312--markbind-master.netlify.app/userguide/components/others#quizzes and the behavior seems to be corrected 👍 |
yucheng11122017
left a comment
There was a problem hiding this comment.
LGTM! Thanks @lesterong
|
LGTM! I've checked out the tests as well and it seems to work as intended. Thanks @lesterong! |
|
@all-contributors please add lesterong for code |
|
I've put up a pull request to add @lesterong! 🎉 |
What is the purpose of this pull request?
Overview of changes:
Only allow the click event on the Next button in Quiz to be triggered once. Resolves #2188.
Anything you'd like to highlight/discuss:
NIL
Testing instructions:
NIL
Proposed commit message: (wrap lines at 72 characters)
Fix Quiz next button skipping questions
Checklist: ☑️