Skip to content

fix: migrate Cloudinary assets to Prismic Media Library - #1777

Merged
jomifepe merged 6 commits into
mainfrom
jp/fix-cloudinary-images
Aug 6, 2026
Merged

fix: migrate Cloudinary assets to Prismic Media Library#1777
jomifepe merged 6 commits into
mainfrom
jp/fix-cloudinary-images

Conversation

@jomifepe

@jomifepe jomifepe commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Replace 17 Cloudinary field icon image URLs with assets in the page-builder-assets repository
  • Replace 1 Cloudinary static fields explainer video URL with a Prismic Media Library asset on prismic-main
  • Serve assets via images.prismic.io without Imgix compression on field icons

Note

Cursor Bugbot is generating a summary for commit 52e4d48. Configure here.

jomifepe and others added 2 commits August 5, 2026 11:02
…ssets

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@jomifepe
jomifepe requested a review from a team as a code owner August 5, 2026 10:26
@vercel

vercel Bot commented Aug 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
slice-machine Ready Ready Preview Aug 5, 2026 11:19am

Request Review

Co-authored-by: Cursor <cursoragent@cursor.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 9868ead. Configure here.

Comment thread packages/slice-machine/src/features/builder/StaticFieldsInfoDialog.tsx Outdated
Remove ?auto=format,compress to preserve original asset quality,
matching what Cloudinary served before migration.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>

@EverybodyToTheLimit EverybodyToTheLimit left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

17 thumbnails + 1 video all 200. Confirmed the changed Video is @prismicio/editor-ui's (takes full-URL src), not cloudinary-react's publicId one so the swap is safe. ✅

@jomifepe jomifepe self-assigned this Aug 6, 2026
@jomifepe
jomifepe merged commit e8e8eb3 into main Aug 6, 2026
35 checks passed
@jomifepe
jomifepe deleted the jp/fix-cloudinary-images branch August 6, 2026 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants