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

Migrate JSHandle APIs to async #1303

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

Migrate JSHandle APIs to async #1303

inancgumus opened this issue May 6, 2024 · 0 comments
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

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.

Tasks

  1. documentation good first issue hacktoberfest
@inancgumus inancgumus mentioned this issue May 6, 2024
17 tasks
@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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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

1 participant