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 issue #1353 : Complete the tour when skipStep is the lastStep #1479

Merged
merged 1 commit into from
Jun 17, 2021

Conversation

thomasguittonneau
Copy link
Contributor

@thomasguittonneau thomasguittonneau commented May 24, 2021

Call the complete function on the last step
Fixes #1353

@RobbieTheWagner
Copy link
Member

@thomasguittonneau this looks good! Would you mind adding a test case as well please?

@RobbieTheWagner
Copy link
Member

@thomasguittonneau we typically make it a rule to only merge PRs with tests, but I will make an exception here.

@RobbieTheWagner RobbieTheWagner merged commit f270d04 into shipshapecode:master Jun 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Event complete isn't called in case of last steps skipped
2 participants