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

Desobekify BrowserContext wait for event options #1512

Merged
merged 7 commits into from
Nov 6, 2024

Conversation

inancgumus
Copy link
Member

@inancgumus inancgumus commented Nov 5, 2024

What?

  • Moves BrowserContext waitForEvent Sobek-option parsing into the mapping layer.
  • Removes common.WaitForEventOptions as it's only used in the mapping.

Why?

See "Desobekifying Sobek transformation" in #1064.

Checklist

  • I have performed a self-review of my code
  • I have added tests for my changes
  • I have commented on my code, particularly in hard-to-understand areas

Related PR(s)/Issue(s)

#1270

@inancgumus inancgumus added refactor stability runtime stability improvements labels Nov 5, 2024
@inancgumus inancgumus self-assigned this Nov 5, 2024
@inancgumus inancgumus changed the base branch from main to desobekify/browser-context-grant-permission-options-parsing November 5, 2024 20:11
@inancgumus inancgumus force-pushed the desobekify/browser-context-waitForEvent-options branch from 738de64 to efb1ad6 Compare November 5, 2024 20:13
@inancgumus inancgumus force-pushed the desobekify/browser-context-grant-permission-options-parsing branch from 6bcbb03 to df900b8 Compare November 5, 2024 20:19
@inancgumus inancgumus force-pushed the desobekify/browser-context-waitForEvent-options branch from efb1ad6 to 1840be2 Compare November 5, 2024 20:22
@inancgumus inancgumus force-pushed the desobekify/browser-context-grant-permission-options-parsing branch from df900b8 to 998bb01 Compare November 5, 2024 20:23
@inancgumus inancgumus force-pushed the desobekify/browser-context-waitForEvent-options branch 2 times, most recently from 733ab54 to b112c6f Compare November 5, 2024 20:24
@inancgumus inancgumus force-pushed the desobekify/browser-context-grant-permission-options-parsing branch from 998bb01 to 74e5505 Compare November 5, 2024 20:59
@inancgumus inancgumus force-pushed the desobekify/browser-context-waitForEvent-options branch 2 times, most recently from 853819b to a1619b9 Compare November 5, 2024 21:10
ankur22
ankur22 previously approved these changes Nov 6, 2024
Copy link
Collaborator

@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.

LGTM 🚀

@inancgumus inancgumus marked this pull request as ready for review November 6, 2024 15:48
Base automatically changed from desobekify/browser-context-grant-permission-options-parsing to main November 6, 2024 15:51
@inancgumus inancgumus dismissed ankur22’s stale review November 6, 2024 15:51

The base branch was changed.

@inancgumus inancgumus force-pushed the desobekify/browser-context-waitForEvent-options branch from a1619b9 to e582707 Compare November 6, 2024 15:52
@inancgumus inancgumus merged commit 4634741 into main Nov 6, 2024
19 of 23 checks passed
@inancgumus inancgumus deleted the desobekify/browser-context-waitForEvent-options branch November 6, 2024 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor stability runtime stability improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants