Skip to content

doc: Fix Pupeteer documentation for waitFroResponse#2632

Merged
Vorobeyko merged 3 commits intocodeceptjs:3.xfrom
kulfoniasty:patch-1
Oct 26, 2020
Merged

doc: Fix Pupeteer documentation for waitFroResponse#2632
Vorobeyko merged 3 commits intocodeceptjs:3.xfrom
kulfoniasty:patch-1

Conversation

@kulfoniasty
Copy link
Copy Markdown
Contributor

@kulfoniasty kulfoniasty commented Oct 24, 2020

The predicate method accepts the response object and not request

Motivation/Description of the PR

Simple doc update. I believe it is a simple copy&paste mistake.

Applicable helpers:

  • WebDriver
  • Puppeteer
  • Nightmare
  • REST
  • FileHelper
  • Appium
  • Protractor
  • TestCafe
  • Playwright

Applicable plugins:

  • allure
  • autoDelay
  • autoLogin
  • customLocator
  • pauseOnFail
  • puppeteerCoverage
  • retryFailedStep
  • screenshotOnFail
  • selenoid
  • stepByStepReport
  • wdio

Type of change

  • 🔥 Breaking changes
  • 🚀 New functionality
  • 🐛 Bug fix
  • 📋 Documentation changes/updates
  • ♨️ Hot fix
  • 🔨 Markdown files fix - not related to source code
  • 💅 Polish 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)

The predicate method accepts the `response` object and not `request`
@DavertMik
Copy link
Copy Markdown
Contributor

@kulfoniasty please update a corresponding in lib/helpers/Puppeteer.js file
Puppeteer.md is generated from docblocks of source code

@kulfoniasty
Copy link
Copy Markdown
Contributor Author

@DavertMik fixed.

@Vorobeyko Vorobeyko merged commit e93c07c into codeceptjs:3.x Oct 26, 2020
@ikeryo1182 ikeryo1182 mentioned this pull request Oct 29, 2020
31 tasks
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