Skip to content

Feat/mux video grab bag#9

Merged
dylanjha merged 7 commits into
muxinc:mainfrom
cjpillsbury:feat/mux-video-grab-bag
Aug 19, 2021
Merged

Feat/mux video grab bag#9
dylanjha merged 7 commits into
muxinc:mainfrom
cjpillsbury:feat/mux-video-grab-bag

Conversation

@cjpillsbury

Copy link
Copy Markdown
Contributor

Adds support for

@cjpillsbury
cjpillsbury requested a review from dylanjha August 18, 2021 13:21
@cjpillsbury cjpillsbury added enhancement New feature or request mux-video Relates to mux-video element labels Aug 18, 2021

@dylanjha dylanjha 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.

Just a few code nit-picks -- overall looks great though

metadata-video-id="This is a more different ID"
metadata-video-title="IM NOT STARWARS YOURE STARWARS"
metadata-viewer-user-id="...starwars"
prefer-native

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.

This looks great 👍🏼

Comment thread packages/mux-video/src/index.ts
Comment thread packages/mux-video/src/index.ts Outdated
@cjpillsbury
cjpillsbury force-pushed the feat/mux-video-grab-bag branch from f43ae67 to d821ec5 Compare August 19, 2021 17:26
@dylanjha
dylanjha merged commit 9b441e2 into muxinc:main Aug 19, 2021
luwes added a commit that referenced this pull request May 6, 2025
* fix: mux-video-ads set up immediately

* fix: mse / native check + add Mux data ads

* feat: add mux-data-keep-session + fix iOS ads
native playback
luwes added a commit that referenced this pull request May 7, 2025
* fix: mux-video-ads set up immediately

* fix: mse / native check + add Mux data ads

* feat: add mux-data-keep-session + fix iOS ads
native playback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request mux-video Relates to mux-video element

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants