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

DOP-5196: save images to directory and update img src #53

Merged
merged 1 commit into from
Dec 17, 2024
Merged

Conversation

seungpark
Copy link
Collaborator

@seungpark seungpark commented Dec 17, 2024

DOP-5196

This PR updates the offline snooty extension to make images available in offline mode.
While going through each HTML file, it looks for DOM nodes with remote sources. If it finds any, it will download the image and save to the public/images directory to make it local, and update the src to the saved file.

Test build log here.

Sample HTML in S3 found here - download the zip, and open a page with remote images and inspect that it is pointing to the local image with a successful render

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for search-manifest-integration failed. Why did it fail? →

Name Link
🔨 Latest commit d4fa272
🔍 Latest deploy log https://app.netlify.com/sites/search-manifest-integration/deploys/67618eed25e9c2000811b830

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for slack-deploy-extension ready!

Name Link
🔨 Latest commit d4fa272
🔍 Latest deploy log https://app.netlify.com/sites/slack-deploy-extension/deploys/67618eedbfba7900085bb248
😎 Deploy Preview https://deploy-preview-53--slack-deploy-extension.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for persistence-module-ext canceled.

Name Link
🔨 Latest commit d4fa272
🔍 Latest deploy log https://app.netlify.com/sites/persistence-module-ext/deploys/67618eed2f4ebf0008bdfb56

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for git-changed-file-extension canceled.

Name Link
🔨 Latest commit d4fa272
🔍 Latest deploy log https://app.netlify.com/sites/git-changed-file-extension/deploys/67618eed3ac0f60008f417a7

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for snooty-cache-extension canceled.

Name Link
🔨 Latest commit d4fa272
🔍 Latest deploy log https://app.netlify.com/sites/snooty-cache-extension/deploys/67618eed13b10500085f6d30

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for site-links-display failed. Why did it fail? →

Name Link
🔨 Latest commit d4fa272
🔍 Latest deploy log https://app.netlify.com/sites/site-links-display/deploys/67618eed4e09710008f5916f

Copy link

netlify bot commented Dec 17, 2024

👷 Deploy Preview for site-links-display processing.

Name Link
🔨 Latest commit d4fa272
🔍 Latest deploy log https://app.netlify.com/sites/site-links-display/deploys/67618eed4e09710008f5916f

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for test-monorepo-redoc failed. Why did it fail? →

Name Link
🔨 Latest commit d4fa272
🔍 Latest deploy log https://app.netlify.com/sites/test-monorepo-redoc/deploys/67618eed27ecd900080dcaa6

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for redirects-and-publish-extension failed. Why did it fail? →

Name Link
🔨 Latest commit d4fa272
🔍 Latest deploy log https://app.netlify.com/sites/redirects-and-publish-extension/deploys/67618eed3ca55600088c8cab

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for redoc-ext-test canceled.

Name Link
🔨 Latest commit d4fa272
🔍 Latest deploy log https://app.netlify.com/sites/redoc-ext-test/deploys/67618eedd79c3d0008cf2578

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for search-manifest-extension failed. Why did it fail? →

Name Link
🔨 Latest commit d4fa272
🔍 Latest deploy log https://app.netlify.com/sites/search-manifest-extension/deploys/67618eedd1f628000838ab65

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for populate-data-extension canceled.

Name Link
🔨 Latest commit d4fa272
🔍 Latest deploy log https://app.netlify.com/sites/populate-data-extension/deploys/67618eedd79c3d0008cf2574

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for offline-snooty ready!

Name Link
🔨 Latest commit d4fa272
🔍 Latest deploy log https://app.netlify.com/sites/offline-snooty/deploys/67618eedc0c12900083c9b60
😎 Deploy Preview https://deploy-preview-53--offline-snooty.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@seungpark seungpark marked this pull request as ready for review December 17, 2024 17:29
Copy link
Collaborator

@branberry branberry left a comment

Choose a reason for hiding this comment

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

LGTM!

@seungpark seungpark merged commit e6df224 into main Dec 17, 2024
29 of 49 checks passed
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