Closed luwes closed 7 months ago
@luwes is attempting to deploy a commit to the Mux Team on Vercel.
A member of the Team first needs to authorize it.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
media-chrome | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Feb 13, 2024 9:40pm |
media-chrome-demo-nextjs | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Feb 13, 2024 9:40pm |
media-chrome-docs | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Feb 13, 2024 9:40pm |
Attention: 2 lines
in your changes are missing coverage. Please review.
Comparison is base (
92d6d7e
) 78.50% compared to head (11ef40e
) 78.55%. Report is 2 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
src/js/media-controller.js | 66.66% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
fixes https://github.com/epicweb-dev/kcdshop/issues/189
the async logic in handleMediaUpdated was causing rootNode to be undefined when trying to add event listeners.
this bug was only apparent when rapidly connecting / disconnecting media like in many modern JS frameworks