-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
perf(web): Convert image assets to webp
& skip flaky tests
#5570
Conversation
✅ Deploy Preview for novu-design ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for dev-web-novu ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
webp
& skip flaky tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❓ question: I see this file is deleted but no replacement webp file is added.
Does it mean we are not using this image anywhere?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's correct! Sorry for not calling that out
@antonjoel82 Shall we merge this? |
@SokratisVidros Could I please get an approval? |
webp
& skip flaky testswebp
& skip flaky tests
webp
& skip flaky testswebp
& skip flaky tests
* perf: Change assets to webp, fix brand redirect * fix: Update new layout comp with webp
* perf: Change assets to webp, fix brand redirect * fix: Update new layout comp with webp
What changed? Why was the change needed?
public/static/images
towebp
to massively reduce sizeValidation