Skip to content

V9 Remove JSONWireProtocol#12006

Merged
christian-bromann merged 12 commits intowebdriverio:v9from
tamil777selvan:v9
Feb 2, 2024
Merged

V9 Remove JSONWireProtocol#12006
christian-bromann merged 12 commits intowebdriverio:v9from
tamil777selvan:v9

Conversation

@tamil777selvan
Copy link
Member

Proposed changes

Fixes #11859

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

Reviewers: @webdriverio/project-committers

erwinheitzman and others added 6 commits December 11, 2023 22:48
* (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>
…iverio#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed
* (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>
…iverio#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed
@tamil777selvan tamil777selvan deleted the branch webdriverio:v9 January 15, 2024 05:57
@tamil777selvan tamil777selvan deleted the v9 branch January 15, 2024 05:57
@tamil777selvan tamil777selvan restored the v9 branch January 15, 2024 05:57
@christian-bromann christian-bromann added this to the v9 milestone Jan 16, 2024
@christian-bromann
Copy link
Member

Thanks for the contribution @tamil777selvan , I think we should calling "code-freeze" on the main branch, create a v8 branch, and continue v9 development on main. With those big changes it will be hard, if not impossible to maintain 2 development branches.

Could you do me a favor and join us on Discord? I would love to invite you to the collaborator channel so we can better coordinate the development.

erwinheitzman and others added 2 commits January 26, 2024 15:07
* (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>
…iverio#11857)

* (webdriverio): merge isDisplayed and isDisplayedWithinViewport

* allow to specify withinViewport check in waitForDisplayed
@christian-bromann
Copy link
Member

@tamil777selvan can you please rebase the PR so we can merge it? I'ld suggest to push your branches directly to this repository so e.g. I could resolve them myself.

@tamil777selvan
Copy link
Member Author

@tamil777selvan can you please rebase the PR so we can merge it? I'ld suggest to push your branches directly to this repository so e.g. I could resolve them myself.

sure @christian-bromann let me do a rebase now.

christian-bromann and others added 2 commits January 26, 2024 19:19
* (webdriverio): make puppeteer a peer dependency

* fix type issues

* fix unit tests
Copy link
Member

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

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

LGTM 👍 thanks for keeping the JSONWP commands around still used by Appium.

@christian-bromann christian-bromann added the PR: Breaking Change 💥 PRs that contain breaking changes label Feb 2, 2024
@christian-bromann christian-bromann merged commit 6aa3d3f into webdriverio:v9 Feb 2, 2024
@tamil777selvan tamil777selvan deleted the v9 branch February 2, 2024 08:13
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>
@allmoatasem
Copy link

@christian-bromann is JSONWP still around in v9.1.0? Our team is running wdio test on android and iOS and driver.setGeoLocation isn't recognised anymore. Tests are failing with error TypeError: driver.setGeoLocation is not a function.

@allmoatasem
Copy link

Got to work by installing the standalone https://github.com/webdriverio-community/wdio-jsonwp-service. Thanks still!

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

None yet

Development

Successfully merging this pull request may close these issues.

4 participants