Skip to content
This repository has been archived by the owner on Jan 30, 2025. It is now read-only.

Migrate JSHandle APIs to async #1303

Closed
5 tasks done
Tracked by #428
inancgumus opened this issue May 6, 2024 · 0 comments
Closed
5 tasks done
Tracked by #428

Migrate JSHandle APIs to async #1303

inancgumus opened this issue May 6, 2024 · 0 comments
Assignees
Labels
async supports async (promises) breaking PRs that need to be mentioned in the breaking changes section of the release notes

Comments

@inancgumus
Copy link
Member

inancgumus commented May 6, 2024

Tasks

Preview Give feedback
  1. mapping refactor stability
    inancgumus
  2. async mapping
    inancgumus
  3. 1 of 1
    documentation good first issue hacktoberfest
    ankur22

What

Migrate the following APIs according to the How section in #428:

Class Method Current k6 Return PW return
JSHandle asElement ElementHandle|null TextendsNode?ElementHandle:null
JSHandle dispose void Promise
JSHandle evaluate<R, Arg> R not found in Playwright
JSHandle evaluateHandle<R, Arg> JSHandle not found in Playwright
JSHandle getProperties Map<string,JSHandle> Promise<Map<string,JSHandle>>
JSHandle jsonValue any Promise

For details on the Why and How, check the overarching issue in #428.

@inancgumus inancgumus added async supports async (promises) breaking PRs that need to be mentioned in the breaking changes section of the release notes labels May 6, 2024
@inancgumus inancgumus self-assigned this Jun 3, 2024
This was referenced Jun 4, 2024
@ankur22 ankur22 self-assigned this Jun 7, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
async supports async (promises) breaking PRs that need to be mentioned in the breaking changes section of the release notes
Projects
None yet
Development

No branches or pull requests

2 participants