Skip to content
This repository has been archived by the owner on Jun 2, 2022. It is now read-only.

chore: Fix Image SSR blink #48

Merged
merged 1 commit into from
Sep 27, 2021
Merged

chore: Fix Image SSR blink #48

merged 1 commit into from
Sep 27, 2021

Conversation

tlgimenes
Copy link
Contributor

What's the purpose of this pull request?

I was trying to fix the wild blink of the SSRed product image on the pdp. However, while digging I've found this issue open on Gatsby and decided to add a todo in our code for us to remember to upgrade it once they fix this problem

How it works?

Sonarqube bot will warn us about it and eventually we will upgrade gatsby-plugin-image and this problem will be fixed

@netlify
Copy link

netlify bot commented Sep 24, 2021

✔️ Deploy Preview for basestore ready!

🔨 Explore the source changes: 820eb1b

🔍 Inspect the deploy log: https://app.netlify.com/sites/basestore/deploys/614fb90579d2cf6a0dad8af4

😎 Browse the preview: https://deploy-preview-48--basestore.netlify.app

@gatsby-cloud
Copy link

gatsby-cloud bot commented Sep 24, 2021

Gatsby Cloud Build Report

basestore

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 2m

Performance

Lighthouse report

Metric Score
Performance 💚 98
Accessibility 💚 100
Best Practices 💚 93
SEO 💚 92

🔗 View full report

@vtex-sites
Copy link

vtex-sites bot commented Sep 24, 2021

Preview is ready

This pull request generated a Preview

👀   Preview: https://preview-48--base.preview.vtex.app
🔬   Go deeper by inspecting the Build Logs
📝   based on commit 24a9d87

@tlgimenes tlgimenes merged commit 24a9d87 into master Sep 27, 2021
@tlgimenes tlgimenes deleted the fix/blink branch September 27, 2021 17:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants