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

feat: Implementation for connecting remote playwright browser #1488

Closed
wants to merge 2 commits into from
Closed

feat: Implementation for connecting remote playwright browser #1488

wants to merge 2 commits into from

Conversation

Archish27
Copy link
Contributor

No description provided.

@ghost
Copy link

ghost commented Jun 5, 2021

CLA assistant check
All CLA requirements met.

/// </summary>
/// <param name="wsEndpoint">Call wsEndpoint</param>
/// <param name="options">Call options</param>
Task<IBrowser> ConnectAsync(string wsEndpoint, BrowserTypeConnectOptions options = default);
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we have an upstream PR for this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hey, nope this is not yet implemented for visibility have raised PR. Currently, it is not working as expected I have been stuck while retrieving the Browser object. It would be great if you guide me through this we can get this connect feature fully implemented as per as requirement. I happy to contribute on this :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kblok Can you please guide me through this or do you any upcoming PRs with such feature implemented?

Copy link
Contributor

Choose a reason for hiding this comment

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

All the code inside the generated directory is auto-generated using generateDotnetApi.js.
So, if you want to generate this connect method, you should change class-browsertype.md first.
Once approved, all the interfaces and options will be auto-generated for you :)

@Archish27
Copy link
Contributor Author

Closing this PR tracking here : #1602

@Archish27 Archish27 closed this Jul 5, 2021
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.

None yet

2 participants