Skip to content

Add Example use for Puppeteer#74

Merged
rorychatt merged 6 commits into
Ivy-Interactive:mainfrom
mohamed95ivy:Feat/51-Example-For-PuppeteerSharp
Sep 29, 2025
Merged

Add Example use for Puppeteer#74
rorychatt merged 6 commits into
Ivy-Interactive:mainfrom
mohamed95ivy:Feat/51-Example-For-PuppeteerSharp

Conversation

@mohamed95ivy

Copy link
Copy Markdown

No description provided.

@mohamed95ivy

Copy link
Copy Markdown
Author
image

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a complete PuppeteerSharp demonstration project that showcases three main browser automation capabilities: taking website screenshots, generating PDFs, and extracting page metadata. The project is built using the Ivy web framework for C# and demonstrates practical implementations of PuppeteerSharp functionality.

  • Implements three demo applications for different PuppeteerSharp use cases
  • Creates a complete project structure with proper configuration and dependencies
  • Provides examples of browser automation including screenshot capture, PDF generation, and metadata extraction

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
README.md Project documentation with setup and deployment instructions
PuppeteerSharpDemo.csproj Project configuration with Ivy and PuppeteerSharp dependencies
Program.cs Application entry point with server configuration
GlobalUsings.cs Global using statements for Ivy framework components
Apps/PuppeteerSharpDemoScreenshot.cs Screenshot capture functionality implementation
Apps/PuppeteerSharpDemoPdf.cs PDF generation from web pages implementation
Apps/PuppeteerSharpDemoMetadata.cs Page metadata extraction implementation
.gitignore Standard .NET project ignore patterns

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread puppeteersharp/Apps/PuppeteerSharpDemoScreenshot.cs Outdated
Comment thread puppeteersharp/Apps/PuppeteerSharpDemoScreenshot.cs Outdated
Comment thread puppeteersharp/Apps/PuppeteerSharpDemoScreenshot.cs Outdated
Comment thread puppeteersharp/Apps/PuppeteerSharpDemoPdf.cs Outdated
Comment thread puppeteersharp/Apps/PuppeteerSharpDemoMetadata.cs Outdated
Comment thread puppeteersharp/Apps/PuppeteerSharpDemoScreenshot.cs
@rorychatt
rorychatt requested a review from Copilot September 25, 2025 09:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread puppeteersharp/Apps/PuppeteerSharpDemoScreenshot.cs Outdated
Comment thread puppeteersharp/Apps/PuppeteerSharpDemoScreenshot.cs Outdated
Comment thread puppeteersharp/Apps/PuppeteerSharpDemoPdf.cs Outdated
@rorychatt

Copy link
Copy Markdown
Collaborator

I could not run the project with dotnet run for some reason. Will try again later

@mohamed95ivy

Copy link
Copy Markdown
Author

@rorychatt any more comments?

@rorychatt

Copy link
Copy Markdown
Collaborator

@mohamed95ivy sorry for running a bit late.

Awesome project.

Minor comment is that it's better if we don't store the files locally and instead make it so user downloads them (so it's stored for the user, not for the server) -> could you implement this?

@rorychatt rorychatt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Left a comment

@mohamed95ivy

Copy link
Copy Markdown
Author

@rorychatt Now, the browser downloads the files.

@rorychatt
rorychatt merged commit 01bc5e3 into Ivy-Interactive:main Sep 29, 2025
4 of 6 checks passed
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.

4 participants