Skip to content

Conversation

KMKoushik
Copy link
Collaborator

@KMKoushik KMKoushik commented Dec 22, 2019

Motivation/Description of the PR

Applicable helpers:

  • Webdriver
  • Puppeteer
  • Nightmare
  • REST
  • Appium
  • Protractor
  • TestCafe

Resolves #2093

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)

@KMKoushik KMKoushik changed the title Add grabTextFromAll method Add grab*FromAll method Dec 22, 2019
@KMKoushik KMKoushik requested a review from DavertMik December 22, 2019 17:16
@KMKoushik
Copy link
Collaborator Author

I have created a new document under webapi folder. But when I try to generate document It also makes changes which I did not change. Maybe it would have been missed in previous merges. (So I did not add any md files in the PR)

Copy link
Contributor

@DavertMik DavertMik left a comment

Choose a reason for hiding this comment

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

Sounds good so far @koushikmohan1996
Please continue

@DavertMik
Copy link
Contributor

Codacy Here is an overview of what got changed by this pull request:

Clones added
============
- lib/helper/Nightmare.js  2
- lib/helper/Protractor.js  6
- lib/helper/TestCafe.js  2
- lib/helper/WebDriver.js  7
- lib/helper/Puppeteer.js  6
         

See the complete overview on Codacy

@KMKoushik KMKoushik marked this pull request as ready for review December 25, 2019 17:00
@KMKoushik
Copy link
Collaborator Author

Should add GrabCss for nightmare helper too after ##2111 merges

@KMKoushik KMKoushik requested a review from Vorobeyko January 6, 2020 13:28
@DavertMik DavertMik added this to the 3.0 milestone Jan 7, 2020
@DavertMik DavertMik added the 3.0 label Jan 7, 2020
@KMKoushik KMKoushik requested a review from DavertMik January 8, 2020 12:20
@KMKoushik KMKoushik requested a review from kobenguyent January 18, 2020 06:02
@DavertMik DavertMik mentioned this pull request Jan 29, 2020
@DavertMik DavertMik changed the base branch from master to codeceptjs-v3.0 February 26, 2020 10:11
@DavertMik DavertMik merged commit f02e8b4 into codeceptjs:codeceptjs-v3.0 Apr 26, 2020
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.

Unify grab* methods accross helpers
3 participants