Skip to content

docs: update autowait limitations example#11828

Merged
christian-bromann merged 1 commit intowebdriverio:mainfrom
rwaskiewicz:patch-1
Dec 8, 2023
Merged

docs: update autowait limitations example#11828
christian-bromann merged 1 commit intowebdriverio:mainfrom
rwaskiewicz:patch-1

Conversation

@rwaskiewicz
Copy link
Contributor

Proposed changes

update the numeric index used in the last javascript example on the page to access the array with index 1. the conditional above it returns false if the length of the array is not 2. accessing an array at elms[2] on a non-sparse array may not be possible in this situation.

I then worked backwards to the previous example, updated the array access there too for continuity.

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

update the numeric index used in the last javascript example on the page to access the array with index 1. the conditional above it returns `false` if the length of the array is not 2. accessing an array at `elms[2]` on a non-sparse array may not be possible in this situation.

I then worked backwards to the previous example, updated the array access there too for continuity.
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.

Great 👁️ for detail 👍

@christian-bromann christian-bromann added the PR: Docs 📝 PRs that contain changes to the documentation label Dec 8, 2023
@christian-bromann christian-bromann merged commit 55cdaad into webdriverio:main Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Docs 📝 PRs that contain changes to the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants