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

Convert all png of sections directory to webp #5923

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

Mandeep56Singh
Copy link

Description

This PR fixes ##5854

Notes for Reviewers
it converts all the png under src/sections directory to webP
and it resolves path

Signed commits

  • [ x] Yes, I signed my commits.

@Mandeep56Singh
Copy link
Author

I am waiting for your feedback

@l5io
Copy link
Contributor

l5io commented Sep 23, 2024

🚀 Preview for commit b11dd86 at: https://66f1dadb46884513f1110169--layer5.netlify.app

@sudhanshutech
Copy link
Member

@Mandeep56Singh will you sign your commits.

Copy link
Member

@sudhanshutech sudhanshutech left a comment

Choose a reason for hiding this comment

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

@Mandeep56Singh did you check how much this saved us space?

Copy link

🚨 Alert! Git Police! We couldn’t help but notice that one or more of your commits is missing a sign-off. A what? A commit sign-off (your email address).

To amend the commits in this PR with your signoff using the instructions provided in the DCO check.

To configure your dev environment to automatically signoff on your commits in the future, see these instructions.


        Be sure to join the community, if you haven't yet and please leave a ⭐ star on the project 😄

1 similar comment
Copy link

🚨 Alert! Git Police! We couldn’t help but notice that one or more of your commits is missing a sign-off. A what? A commit sign-off (your email address).

To amend the commits in this PR with your signoff using the instructions provided in the DCO check.

To configure your dev environment to automatically signoff on your commits in the future, see these instructions.


        Be sure to join the community, if you haven't yet and please leave a ⭐ star on the project 😄

vishalvivekm and others added 14 commits September 25, 2024 23:09
Signed-off-by: Vivek Vishal <[email protected]>
Signed-off-by: Vivek Vishal <[email protected]>
Signed-off-by: Vivek Vishal <[email protected]>
Signed-off-by: Sudhanshu Dasgupta <[email protected]>
Signed-off-by: Sudhanshu Dasgupta <[email protected]>
Signed-off-by: Sudhanshu Dasgupta <[email protected]>
Signed-off-by: Lee Calcote <[email protected]>
Signed-off-by: Lee Calcote <[email protected]>
Signed-off-by: Yaten Dhingra <[email protected]>
Bumps [@loadable/component](https://github.com/gregberge/loadable-components) from 5.15.3 to 5.16.4.
- [Release notes](https://github.com/gregberge/loadable-components/releases)
- [Changelog](https://github.com/gregberge/loadable-components/blob/main/CHANGELOG.md)
- [Commits](gregberge/loadable-components@v5.15.3...v5.16.4)

---
updated-dependencies:
- dependency-name: "@loadable/component"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Mandeep56Singh <[email protected]>
@Mandeep56Singh
Copy link
Author

Mandeep56Singh commented Sep 25, 2024

@sudhanshutech looks like I maded a mistake 😅 in commit history
I forget to update the branch, and simply push the commit history

is it a problem ?
is there a way to fix it ?

@Mandeep56Singh
Copy link
Author

@sudhanshutech yes space has saved
from 9.3mb to 1.4mb
Before
Screenshot (485)

after
Screenshot (484)

@l5io
Copy link
Contributor

l5io commented Sep 25, 2024

🚀 Preview for commit d7d8699 at: https://66f451d19b40483e1f18f91c--layer5.netlify.app

@vishalvivekm
Copy link
Member

Thank you for your contribution @Mandeep56Singh
Let's discuss this during the website call on Monday at 5:30 PM IST (7:00 AM CT).

Please add it as an agenda item to the meeting minutes.

@Mandeep56Singh
Copy link
Author

@vishalvivekm sorry, I wasn't able to attend the meeting, please tell if this pr need changes. I am not able to see the changes requested by @sudhanshutech.

@l5io
Copy link
Contributor

l5io commented Oct 2, 2024

🚀 Preview for commit 01a808e at: https://66fd31a32515e4d7ee2ba398--layer5.netlify.app

@sudhanshutech
Copy link
Member

@Mandeep56Singh you might have rebased the commit please revert those changes. Either you can create a new pr and closing this one and in the new pr update to webp in chunks and not every images in same pr.

@sudhanshutech
Copy link
Member

@Mandeep56Singh ?

@Mandeep56Singh
Copy link
Author

@sudhanshutech looks like I maded a mistake 😅 in commit history I forget to update the branch, and simply push the commit history

is it a problem ? is there a way to fix it ?

@sudhanshutech please tell me, how to fix the mistake I created.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

6 participants