Skip to content

Updated props to optional for getText and getLocation commands#12308

Merged
christian-bromann merged 3 commits intowebdriverio:mainfrom
jemishgopani:optional-props-doc-main
Feb 20, 2024
Merged

Updated props to optional for getText and getLocation commands#12308
christian-bromann merged 3 commits intowebdriverio:mainfrom
jemishgopani:optional-props-doc-main

Conversation

@jemishgopani
Copy link
Copy Markdown
Contributor

Proposed changes

Fixes #12173

  • Make props as optional parameter for getText and getLocation 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)

Reviewers: @webdriverio/project-committers

Copy link
Copy Markdown
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.

Can we have a test for this in /tests/typings to ensure that this works?

@jemishgopani jemishgopani marked this pull request as draft February 20, 2024 03:57
@christian-bromann
Copy link
Copy Markdown
Member

@jemishgopani can we update the PR based on #12307 ?

@jemishgopani
Copy link
Copy Markdown
Contributor Author

@jemishgopani can we update the PR based on #12307 ?

Sure

@christian-bromann christian-bromann marked this pull request as ready for review February 20, 2024 16:36
Copy link
Copy Markdown
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.

👍

@christian-bromann christian-bromann added the PR: Bug Fix 🐛 PRs that contain bug fixes label Feb 20, 2024
@christian-bromann christian-bromann merged commit e44fb03 into webdriverio:main Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Bug Fix 🐛 PRs that contain bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🐛 Bug]: getSize() typing expects at least 1 argument

2 participants