Skip to content

fix: add async effect for loading media - #492

Merged
luwes merged 6 commits into
muxinc:mainfrom
luwes:async-load
Nov 8, 2022
Merged

fix: add async effect for loading media#492
luwes merged 6 commits into
muxinc:mainfrom
luwes:async-load

Conversation

@luwes

@luwes luwes commented Nov 5, 2022

Copy link
Copy Markdown
Contributor
  • this adds an async effect for init playback core and loading the media for the mux-video and mux-audio custom elements.
  • made the mediaEl required in playback-core making some code and types easier.

@luwes luwes self-assigned this Nov 5, 2022
@luwes
luwes requested a review from a team as a code owner November 5, 2022 01:30
@vercel

vercel Bot commented Nov 5, 2022

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
elements-demo-create-react-app ✅ Ready (Inspect) Visit Preview Nov 8, 2022 at 5:27PM (UTC)
elements-demo-nextjs ✅ Ready (Inspect) Visit Preview Nov 8, 2022 at 5:27PM (UTC)
elements-demo-svelte-kit ✅ Ready (Inspect) Visit Preview Nov 8, 2022 at 5:27PM (UTC)
elements-demo-vanilla ✅ Ready (Inspect) Visit Preview Nov 8, 2022 at 5:27PM (UTC)
elements-demo-vue ✅ Ready (Inspect) Visit Preview Nov 8, 2022 at 5:27PM (UTC)

@codecov

codecov Bot commented Nov 5, 2022

Copy link
Copy Markdown

Codecov Report

Merging #492 (5b48902) into main (81974ce) will increase coverage by 0.07%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #492      +/-   ##
==========================================
+ Coverage   81.48%   81.55%   +0.07%     
==========================================
  Files          45       45              
  Lines        6460     6464       +4     
  Branches      279      279              
==========================================
+ Hits         5264     5272       +8     
+ Misses       1194     1190       -4     
  Partials        2        2              
Impacted Files Coverage Δ
packages/mux-video/src/index.ts 65.94% <70.58%> (+0.34%) ⬆️
packages/mux-audio/src/index.ts 65.30% <76.00%> (+0.49%) ⬆️
packages/playback-core/src/index.ts 73.55% <85.71%> (+0.41%) ⬆️
packages/playback-core/src/autoplay.ts 88.75% <100.00%> (+0.47%) ⬆️
packages/playback-core/src/preload.ts 100.00% <100.00%> (+1.31%) ⬆️

@gkatsev gkatsev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Seems like a reasonable approach.

Comment thread packages/mux-video/src/index.ts Outdated
Comment thread packages/mux-video/src/index.ts
Comment thread packages/mux-video/src/index.ts

@cjpillsbury cjpillsbury left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@luwes
luwes merged commit d2b250a into muxinc:main Nov 8, 2022
@luwes
luwes deleted the async-load branch November 8, 2022 18:53
@github-actions github-actions Bot mentioned this pull request Sep 9, 2025
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.

3 participants