-
Notifications
You must be signed in to change notification settings - Fork 47
chore: reduce cloudinary usage #858
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
Conversation
🦋 Changeset detectedLatest commit: 35b0ccc The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
5 Skipped Deployments
|
Reduced usage of Cloudinary in the application.
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.
Pull Request Overview
This PR reduces Cloudinary bandwidth usage by removing unused assets, replacing a video with static images, and optimizing large images that consume significant bandwidth.
- Removed 400+ unused image, video, and icon asset types from the asset type definitions
- Replaced animated video elements with static frame images in the hands section of the homepage
- Optimized three high-bandwidth images (homepage hero and jobs page images) and updated references to use
-optimizedversions
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| apps/status.app/src/app/_components/assets/types.ts | Removed unused asset type definitions and added three new optimized image variants |
| apps/status.app/src/app/(website)/page.tsx | Updated homepage hero image reference to use optimized variant |
| apps/status.app/src/app/(website)/jobs/page.tsx | Updated jobs page images to use optimized variants |
| apps/status.app/src/app/(website)/_components/hands-section.tsx | Replaced Video components with Image components using static frames to reduce bandwidth |
| .changeset/seven-olives-design.md | Added changeset documenting the Cloudinary usage reduction |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
apps/status.app/src/app/(website)/_components/hands-section.tsx
Outdated
Show resolved
Hide resolved
felicio
left a comment
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.
👍 as hotfix but will follow-up in the issue
|
@jkbktl btw, did remove the assets from the platform itself? |
|
I could still access it directly. |
JulesFiliot
left a comment
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.
![]()

relates https://github.com/status-im/status-website/issues/1624
Video replacement discussed with @sunleos
Other changes:
apps/portfolioas it's missingpackages/status-networkas it's not using cloudinary at allstatus.app,apps/portfolioandpackages/walletPossible improvements in the future: