Skip to content

Conversation

@jonatansberg
Copy link
Member

no ref

Update video path and include the video assets in Vite asset pipeline to ensure we serve the correct assets across different environments.

Previously we were successfully serving the existing assets from Ember, but at some point that seemed to have broken in production.

no ref 

Update video path and include the video assets in Vite asset pipeline to
ensure we serve the correct assets across different environments.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 2, 2026

Walkthrough

The change modifies a single video asset path in the admin index.html file. The loading indicator's video source reference was updated from a relative path ("assets/videos/logo-loader.mp4") to an absolute path from the site root ("/src/assets/videos/logo-loader.mp4"). No structural, behavioral, or exported entity changes are present.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: fixing a logo loading animation issue in production by updating the video asset path.
Description check ✅ Passed The description is related to the changeset, explaining the video path update and its purpose to fix production asset serving.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-logo-loading-animation

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@jonatansberg jonatansberg merged commit 340135b into main Feb 2, 2026
35 checks passed
@jonatansberg jonatansberg deleted the fix-logo-loading-animation branch February 2, 2026 08:29
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