Skip to content
This repository has been archived by the owner on Dec 16, 2023. It is now read-only.

Fix #927 - document .wait #961

Closed
wants to merge 1 commit into from
Closed

Fix #927 - document .wait #961

wants to merge 1 commit into from

Conversation

kbariotis
Copy link

Indeed, wait function is a neat one especially when messing with a lot of AJAX requests in your SUT. It should be present in the docs. We can also add some examples if you would like.

@assaf
Copy link
Owner

assaf commented Oct 31, 2015

Sweet. Can you add something about promises, though? This method (like most others), either called with a callback, or returns a promise you have to wait on.

@kbariotis kbariotis changed the title Fix #927 Fix #927 - document .wait Oct 31, 2015
@kbariotis
Copy link
Author

@assaf looks good?

@assaf
Copy link
Owner

assaf commented Oct 31, 2015

options (required): Options that determine how long to wait (see below)

It can't be both optional and required

@kbariotis
Copy link
Author

@assaf (sorry, that moment when you're trying to help and you're doing it worst!) updated and also clean up history.

Thank you

@assaf assaf closed this Apr 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants