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.

Sign up for GitHub

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

Jump to bottom

Fix lp 55 profile picture compression #46

Merged
merged 5 commits into from
Aug 10, 2023
Merged

Fix lp 55 profile picture compression #46

merged 5 commits into from
Aug 10, 2023

Conversation

Charvit123
Copy link
Collaborator

What

Please include a summary of the changes.

  • set image compression size of the image if it is up to 7mb it reduces to nearly about 500kb and above 7mb it gives an error

How

  • adds npm package browser-image-compression

Why

to improve storage capacity

Checklist

  • My code follows the style guidelines of this project and is properly formatted
  • I have performed a self-review of my own changes
  • All existing tests pass, and I have added new ones if required

amanramoliya
amanramoliya approved these changes Aug 10, 2023
View reviewed changes
Copy link
Collaborator

@amanramoliya amanramoliya left a comment

Choose a reason for hiding this comment

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

Ready to merge.

@Charvit123 Charvit123 merged commit 839411f into main Aug 10, 2023
9 checks passed
@Charvit123 Charvit123 deleted the fix-LP-55-profile-picture-compression branch August 10, 2023 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amanramoliya amanramoliya amanramoliya approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Charvit123 @amanramoliya