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

Add window.prompt stubbing example to api section #294

Merged
merged 2 commits into from
Dec 2, 2017

Conversation

microbouji
Copy link
Contributor

This addresses #193. I followed the pattern of the "Stubbing window.fetch" example recipe to make the change before the app runs.

Also, thank you folks for being friendly to new contributors. The "first-timers-only" label of this issue is what made the difference for me between starting 24 pull requests or not.

@CLAassistant
Copy link

CLAassistant commented Dec 1, 2017

CLA assistant check
All committers have signed the CLA.

@jennifer-shehane jennifer-shehane self-requested a review December 1, 2017 22:12
Copy link
Member

@jennifer-shehane jennifer-shehane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the added example!

Could you possibly remove the semicolons? We're one of those 'no-semicolon' projects. 😬

@microbouji
Copy link
Contributor Author

Sorry about missing that! Fixed it.

@bahmutov bahmutov merged commit 59f8b3c into cypress-io:develop Dec 2, 2017
@bahmutov
Copy link
Contributor

bahmutov commented Dec 2, 2017

great example, thank you @microbouji

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.

4 participants