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

chore: Improve useStorage and validateCart performance #416

Merged
merged 1 commit into from
Mar 25, 2022

Conversation

igorbrasileiro
Copy link
Contributor

@igorbrasileiro igorbrasileiro commented Mar 23, 2022

What's the purpose of this pull request?

Description here: vtex/faststore#1186

How does it work?

Tell us the role of the new feature, or component, in its context.

How to test it?

Describe the steps with bullet points. Is there any external reference, link, or example?

References

Spread the knowledge: is this any content you used to create this PR that is worth sharing?

Extra tip: add references to related issues or mention people important to this PR may be good for the documentation and reviewing process

Checklist

You may erase this after checking them all ;)

  • [] CHANGELOG entry added

@netlify
Copy link

netlify bot commented Mar 23, 2022

Deploy Preview for basestore ready!

Name Link
🔨 Latest commit 232707f
🔍 Latest deploy log https://app.netlify.com/sites/basestore/deploys/623df381930abf00097563f1
😎 Deploy Preview https://deploy-preview-416--basestore.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 settings.

@vtex-sites
Copy link

vtex-sites bot commented Mar 23, 2022

Preview is ready

This pull request generated a Preview

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

@gatsby-cloud
Copy link

gatsby-cloud bot commented Mar 23, 2022

Gatsby Cloud Build Report

basestore

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

Build Details

View the build logs here.

🕐 Build time: 13m

Performance

Lighthouse report

Metric Score
Performance 💚 94
Accessibility 💚 100
Best Practices 💚 100
SEO 💚 93

🔗 View full report

@igorbrasileiro igorbrasileiro marked this pull request as ready for review March 23, 2022 20:03
@igorbrasileiro igorbrasileiro requested a review from a team March 23, 2022 20:03
@igorbrasileiro igorbrasileiro force-pushed the chore/improve-use-storage branch 4 times, most recently from fcd1ccb to a41e888 Compare March 25, 2022 12:12
@igorbrasileiro
Copy link
Contributor Author

igorbrasileiro commented Mar 25, 2022

Pay attention the result of lhci for PLP page, isn't right.
Ran 20 times each page
Page result - https://base.vtex.app/

Metric Mean Standard deviation
Cumulative Layout shift (CLS) 0.00 0.00
First Contentful Paint (FCP) 1719.76 5019.47
First Contentful Paint 3g (FCP) 3422.26 20275.96
Largest Contentful Paint (LCP) 1746.17 11413.61
Time to Interactive (TTI) 4000.50 17211.76
Total Blocking Time (TBT) 655.64 10900.26
Performance score 0.812 0.000569
JavaScript Execution Time 2170.80 117899.31
Speed Index 2121.13 35071.75

Page result - https://sfj-f054c7b--base.preview.vtex.app/

Metric Mean Standard deviation
Cumulative Layout shift (CLS) 0.00 0.00
First Contentful Paint (FCP) 1768.90 5165.54
First Contentful Paint 3g (FCP) 3614.17 7.41
Largest Contentful Paint (LCP) 1949.98 19887.27
Time to Interactive (TTI) 3543.17 17954.36
Total Blocking Time (TBT) 380.10 706.44
Performance score 0.879 0.000189
JavaScript Execution Time 1543.65 9308.69
Speed Index 2061.99 4498.76

@igorbrasileiro igorbrasileiro force-pushed the chore/improve-use-storage branch from a41e888 to f054c7b Compare March 25, 2022 15:51
@igorbrasileiro igorbrasileiro changed the title chore: Bump @faststore/sdk chore: Improve useStorage and validateCart performance Mar 25, 2022
@igorbrasileiro igorbrasileiro force-pushed the chore/improve-use-storage branch from f054c7b to 232707f Compare March 25, 2022 16:53
@igorbrasileiro igorbrasileiro merged commit cdcec58 into master Mar 25, 2022
@igorbrasileiro igorbrasileiro deleted the chore/improve-use-storage branch March 25, 2022 17:21
@igorbrasileiro igorbrasileiro self-assigned this Mar 31, 2022
@igorbrasileiro igorbrasileiro added the performance Improvements to performance label Mar 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
performance Improvements to performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant