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(plugin-percy): add option to delay pupeteer screenshot and modif… #559

Merged
merged 3 commits into from
Jul 6, 2023

Conversation

noel-schenk
Copy link
Contributor

@noel-schenk noel-schenk commented Jul 4, 2023

Description

Pupeteer needs sometimes more time to render a page. There are multiple ways to solve this, in this PR there are 2 ways implemented. One option to wait x seconds and one options to listen for a network change using pupeteers native options.

#485
#373
#357

What is the purpose of this pull request?

  • Bug fix
  • New Feature

@stackblitz
Copy link

stackblitz bot commented Jul 4, 2023

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@codesandbox
Copy link

codesandbox bot commented Jul 4, 2023

This branch is running in CodeSandbox. Use the links below to review this PR faster.


CodeSandbox logoCodeSandbox logo  Open in CodeSandbox Web Editor | VS Code | VS Code Insiders | Preview

@netlify
Copy link

netlify bot commented Jul 4, 2023

Deploy Preview for histoire-examples-svelte3 ready!

Name Link
🔨 Latest commit 7766b83
🔍 Latest deploy log https://app.netlify.com/sites/histoire-examples-svelte3/deploys/64a694c1b77cc600082311d3
😎 Deploy Preview https://deploy-preview-559--histoire-examples-svelte3.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Jul 4, 2023

Deploy Preview for histoire-site ready!

Name Link
🔨 Latest commit 7766b83
🔍 Latest deploy log https://app.netlify.com/sites/histoire-site/deploys/64a694c1a872fe00089edd9e
😎 Deploy Preview https://deploy-preview-559--histoire-site.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Jul 4, 2023

Deploy Preview for histoire-examples-vue3 ready!

Name Link
🔨 Latest commit 7766b83
🔍 Latest deploy log https://app.netlify.com/sites/histoire-examples-vue3/deploys/64a694c10a15c100089e2630
😎 Deploy Preview https://deploy-preview-559--histoire-examples-vue3.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Jul 4, 2023

Deploy Preview for histoire-controls ready!

Name Link
🔨 Latest commit 7766b83
🔍 Latest deploy log https://app.netlify.com/sites/histoire-controls/deploys/64a694c19f793b0008cad7e5
😎 Deploy Preview https://deploy-preview-559--histoire-controls.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Akryum
Copy link
Member

Akryum commented Jul 6, 2023

Looks like ESLint throws errors

@noel-schenk
Copy link
Contributor Author

@Akryum Yes sorry about that. They are fixed now.

@Akryum Akryum merged commit e6477c4 into histoire-dev:main Jul 6, 2023
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.

3 participants