Skip to content

fix: pass context_id instead of browser_id to BrowserContext.close#33

Open
itq148 wants to merge 1 commit intoftes:mainfrom
itq148:main
Open

fix: pass context_id instead of browser_id to BrowserContext.close#33
itq148 wants to merge 1 commit intoftes:mainfrom
itq148:main

Conversation

@itq148
Copy link
Copy Markdown

@itq148 itq148 commented Mar 24, 2026

I happened to found this little typo/bug.
Although it's a minor issue it's quite misleading.
Actually I passed browser_id, not context_id.

Copy link
Copy Markdown
Owner

@ftes ftes left a comment

Choose a reason for hiding this comment

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

Thanks, excellent!

@ftes ftes enabled auto-merge (squash) March 24, 2026 13:52
@ftes
Copy link
Copy Markdown
Owner

ftes commented Mar 27, 2026

Hey @itq148, if you could re-push with a signed commit that would be excellent.
Otherwise I'll take care of that soon.

@itq148
Copy link
Copy Markdown
Author

itq148 commented Apr 1, 2026

I'll take care of it within 24 hours. Thanks for the notice!

auto-merge was automatically disabled April 2, 2026 10:52

Head branch was pushed to by a user without write access

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants