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

(webdriverio): merge isDisplayed and isDisplayedWithinViewport #11857

Merged
merged 2 commits into from Dec 12, 2023

Conversation

christian-bromann
Copy link
Member

Proposed changes

To keep the API clean we prefer one command with strictness options over multiple commands.

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Checklist

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • I have added proper type definitions for new commands (if appropriate)

Further comments

n/a

Reviewers: @webdriverio/project-committers

Copy link
Member

@erwinheitzman erwinheitzman left a comment

Choose a reason for hiding this comment

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

Nice one, LGTM 👍

@christian-bromann christian-bromann merged commit 855e25e into v9 Dec 12, 2023
12 of 14 checks passed
@christian-bromann christian-bromann deleted the cb/merge-displayed-check-commands branch December 12, 2023 16:08
christian-bromann added a commit that referenced this pull request Dec 28, 2023
* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed
christian-bromann added a commit that referenced this pull request Jan 26, 2024
* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed
christian-bromann added a commit that referenced this pull request Jan 28, 2024
* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed
christian-bromann added a commit that referenced this pull request Feb 2, 2024
* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* V9 remove jsonWireProtocol

* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* (internal): update some dependencies that couldn't be updated due to support for Node.js v16

closes #11938
fixes #11858

* (webdriverio): make puppeteer a peer dependency (#12103)

* (webdriverio): make puppeteer a peer dependency

* fix type issues

* fix unit tests

---------

Co-authored-by: Erwin Heitzman <15839059+erwinheitzman@users.noreply.github.com>
Co-authored-by: Christian Bromann <git@bromann.dev>
christian-bromann added a commit that referenced this pull request Feb 14, 2024
* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* Migrating from got to fetch

* Migrate got to fetch in packages

* migrate got to fetch & update unit test

* doc update for proxy setup

* rebase and migrate got to fetch in browserstack

* Migrate hostname from 0.0.0.0 to localhost

* fix unit test by removing headers in safari driver

* fix smoke test: update nock to beta

* fix launch test for firefox

* fix unit test for startGeckodriver

* fix firefox executions in node 18

* Incorporating review comments

* fix component tests

* cleaning up RequestLibOptions

* Removing RequestLibOptions & incorporating other review comments

* use process from globalThis

---------

Co-authored-by: Erwin Heitzman <15839059+erwinheitzman@users.noreply.github.com>
Co-authored-by: Christian Bromann <git@bromann.dev>
christian-bromann added a commit that referenced this pull request Feb 17, 2024
* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed
christian-bromann added a commit that referenced this pull request Feb 17, 2024
* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* V9 remove jsonWireProtocol

* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* (internal): update some dependencies that couldn't be updated due to support for Node.js v16

closes #11938
fixes #11858

* (webdriverio): make puppeteer a peer dependency (#12103)

* (webdriverio): make puppeteer a peer dependency

* fix type issues

* fix unit tests

---------

Co-authored-by: Erwin Heitzman <15839059+erwinheitzman@users.noreply.github.com>
Co-authored-by: Christian Bromann <git@bromann.dev>
christian-bromann added a commit that referenced this pull request Feb 17, 2024
* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* V9 drop support for Node.js 16 (#11493)

* (internal): drop support for Node.js 16

update engines of package.json files

* (internal): drop support for Node.js 16

update CONTRIBUTING.md with Node.js 20 as recommendation

* (internal): drop support for Node.js 16

update github workflows

* (internal): drop support for Node.js 16

update external packages that dropped the support already

* (internal): drop support for Node.js 16

resolve PR feedback

* Update packages/webdriver/package.json

* Update packages/wdio-cucumber-framework/package.json

* Update package.json

---------

Co-authored-by: Christian Bromann <git@bromann.dev>

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport (#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed

* Migrating from got to fetch

* Migrate got to fetch in packages

* migrate got to fetch & update unit test

* doc update for proxy setup

* rebase and migrate got to fetch in browserstack

* Migrate hostname from 0.0.0.0 to localhost

* fix unit test by removing headers in safari driver

* fix smoke test: update nock to beta

* fix launch test for firefox

* fix unit test for startGeckodriver

* fix firefox executions in node 18

* Incorporating review comments

* fix component tests

* cleaning up RequestLibOptions

* Removing RequestLibOptions & incorporating other review comments

* use process from globalThis

---------

Co-authored-by: Erwin Heitzman <15839059+erwinheitzman@users.noreply.github.com>
Co-authored-by: Christian Bromann <git@bromann.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Breaking Change 💥 PRs that contain breaking changes v9
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants