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

postStepCallback causing a step in the demo to be skipped. #94

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bradhuffman
Copy link
Contributor

Not sure what the intention was, but the example of calling a
postStepCallback was causing step 4 to be skipped. Step 4 demonstrated
a step being displayed as a modal, so my thought is you'd want to see
that. Changed the demo to show an alert between steps demonstrating the
capability of a postStepCallback.

Not sure what the intention was, but the example of calling a
postStepCallback was causing step 4 to be skipped.  Step 4 demonstrated
a step being displayed as a modal, so my thought is you'd want to see
that.  Changed the demo to show an alert between steps demonstrating the
capability of a postStepCallback.
@baohouse
Copy link

Related to issue #91.

@mrsweaters
Copy link
Contributor

Could you update your pull request?

@bradhuffman
Copy link
Contributor Author

Not sure what you mean by update my pull request. Close it?

@mrsweaters
Copy link
Contributor

@bradhuffman Sorry, merge master into your branch so that it can be automagically merged in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants