Copy built album-embed assets into Docker images#4084
Copy built album-embed assets into Docker images#4084ildyria merged 1 commit intoLycheeOrg:masterfrom
Conversation
The embed widget is built to public/embed/ (which is .gitignored) but only public/build/ was copied from the node build stage. This caused /embed/lychee-embed.js to fall through to Laravel routing instead of being served as a static file. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
📝 WalkthroughWalkthroughBoth Dockerfile and Dockerfile-legacy are updated to copy the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. No actionable comments were generated in the recent review. 🎉 Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
FWIW, this commit is currently built & deployed to https://pictures.dzombak.com; a working embedded album is visible at https://www.dzombak.com/blog/2026/02/birds-of-sanibel-island/ . |
The album/stream embed widget is built to
public/embed/(which is .gitignored) but onlypublic/build/was copied from the node build stage. This caused embedded albums to be broken in Lychee 7.x.Summary by CodeRabbit
public/embedare now available at runtime alongside existing public build assets, improving asset delivery capabilities.