Skip to content

Conversation

DavertMik
Copy link
Contributor

This reverts commit 4a769f1.

Previous PR broke backwards compatibility.

We will change how grab* methods work but once we will be releasing 3.0

Motivation/Description of the PR

Applicable helpers:

  • [ x ] Webdriver

  • Puppeteer

  • Nightmare

  • REST

  • Appium

  • Protractor

  • TestCafe

  • Description of this PR, which problem it solves

  • A link to the corresponding issue (if applicable).

Type of change

  • Breaking changes
  • New functionality
  • Bug fix
  • Documentation changes/updates
  • Hot fix
  • Markdown files fix - not related to source code

Checklist:

  • Tests have been added
  • Documentation has been added (Run npm run docs)
  • Lint checking (Run npm run lint)
  • Local tests are passed (Run npm test)

@elukoyanov
Copy link
Collaborator

I think, we should no revert everything, just make PR with previous signature

@DavertMik
Copy link
Contributor Author

grab* will be fixed in #2098

@DavertMik DavertMik merged commit 88d2d9f into master Jan 8, 2020
@DavertMik DavertMik deleted the revert-2054-fix-webdriver-return-values branch January 8, 2020 11:55
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.

2 participants