Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: usage on rendition change #1088

Merged
merged 4 commits into from
Mar 30, 2021
Merged

feat: usage on rendition change #1088

merged 4 commits into from
Mar 30, 2021

Conversation

brandonocasey
Copy link
Contributor

Description

Switch all this.masterPlaylistLoader.media_ calls to this.switchMedia_ which will log the cause behind the switch as a triggered usage event.

@brandonocasey brandonocasey force-pushed the chore/rendition-usage branch from 9ccb581 to f44cb93 Compare March 4, 2021 16:21
@gkatsev gkatsev changed the title chore: usage on rendition change fix: usage on rendition change Mar 4, 2021
@gkatsev gkatsev changed the title fix: usage on rendition change feat: usage on rendition change Mar 4, 2021
@codecov
Copy link

codecov bot commented Mar 30, 2021

Codecov Report

Merging #1088 (2b47356) into main (de0b55b) will increase coverage by 0.01%.
The diff coverage is 92.85%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1088      +/-   ##
==========================================
+ Coverage   86.06%   86.08%   +0.01%     
==========================================
  Files          38       38              
  Lines        8923     8931       +8     
  Branches     1998     2001       +3     
==========================================
+ Hits         7680     7688       +8     
  Misses       1243     1243              
Impacted Files Coverage Δ
src/master-playlist-controller.js 93.85% <92.85%> (+0.06%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de0b55b...2b47356. Read the comment docs.

@brandonocasey brandonocasey merged commit 1b990f1 into main Mar 30, 2021
@brandonocasey brandonocasey deleted the chore/rendition-usage branch March 30, 2021 16:53
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