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

Improve browser Touchscreen and tap #1581

Merged
merged 4 commits into from
May 8, 2024
Merged

Improve browser Touchscreen and tap #1581

merged 4 commits into from
May 8, 2024

Conversation

inancgumus
Copy link
Member

@inancgumus inancgumus commented May 6, 2024

What?

Improve the async Tap documentation by adding the missing docs.

This might be merged after https://github.com/grafana/k6/releases/tag/v0.51.0. We need to apply the changes to version 0.51.

Checklist

  • I have used a meaningful title for the PR.
  • I have described the changes I've made in the "What?" section above.
  • I have performed a self-review of my changes.
  • I have run the npm start command locally and verified that the changes look good.
  • I have made my changes in the docs/sources/next folder of the documentation.
  • I have reflected my changes in the docs/sources/v{most_recent_release} folder of the documentation.
  • I have reflected my changes in the relevant folders of the two previous k6 versions of the documentation (if still applicable to previous versions).

Related PR(s)/Issue(s)

Updates grafana/xk6-browser#1293

@inancgumus inancgumus added the Area: browser The browser module label May 6, 2024
@inancgumus inancgumus self-assigned this May 6, 2024
@inancgumus inancgumus force-pushed the improve/browser-tap branch from 2320e94 to 931e541 Compare May 6, 2024 12:02
@inancgumus inancgumus force-pushed the improve/browser-tap branch from 931e541 to 4fcc688 Compare May 6, 2024 12:26
Moves the example from page to the touchscreen.tap to prevent redundancy
and improve maintainability of the docs
@inancgumus inancgumus force-pushed the improve/browser-tap branch from 4fcc688 to 1b81abc Compare May 6, 2024 12:29
@inancgumus inancgumus marked this pull request as ready for review May 6, 2024 12:43
@inancgumus inancgumus requested a review from heitortsergent as a code owner May 6, 2024 12:43
@inancgumus inancgumus requested a review from ankur22 May 6, 2024 12:43
@inancgumus
Copy link
Member Author

inancgumus commented May 6, 2024

@heitortsergent @ankur22 Could you look at the current changes? I might apply the changes also to v0.51 after k6's release if the changes are OK.

Copy link
Contributor

@ankur22 ankur22 left a comment

Choose a reason for hiding this comment

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

Overall looks good to me. Some minor edits left for you to consider.

@inancgumus inancgumus merged commit e69c7f6 into main May 8, 2024
5 checks passed
@inancgumus inancgumus deleted the improve/browser-tap branch May 8, 2024 12:43
@inancgumus inancgumus mentioned this pull request May 8, 2024
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: browser The browser module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants