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.

Sign up for GitHub

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

Jump to bottom

Feature/cmcd settings from manifest #13

Open
wants to merge 36 commits into
base: development
Choose a base branch
from
Open

Feature/cmcd settings from manifest #13

wants to merge 36 commits into from

Conversation

matiasrb97
Copy link

No description provided.

santimintegui and others added 30 commits March 11, 2024 16:13
@santimintegui @matiasrb97
Parse ClientDataReporting from manifest file
b34b476
@santimintegui @matiasrb97
add applyCMCDParameters
502277e
@matiasrb97
Using the CMCDParameters configuration under ClientDataReporting in t…
d5bcee5 
…he manifest file
@matiasrb97
Check if cmcd is enabled by checking mdp and player settings. Using m…
52567d7 
…dp over the player settings but when applyCMCDParameters is false, player settings will be used
@santimintegui @matiasrb97
apply serviceLocation filter to data reporting
efd2b05
@santimintegui @matiasrb97
add adapter set filter
957aa2d
@santimintegui @matiasrb97
improve apply filters
28f82bd
@santimintegui @matiasrb97
improve apply filters
ca8725e
@santimintegui @matiasrb97
delete unnecessary atribute in client data reporting
d8d2f0e
@matiasrb97
Change getCmcdParametersMDP to getCmcdParametersFromManifest
938f613
@santimintegui @matiasrb97
add client data reporting model
050b528
@santimintegui @matiasrb97
refactor client data reporting model
a746fa5
@santimintegui @matiasrb97
implemented include in request filter
99d77cc
@santimintegui @matiasrb97
add none include request case
aa23f6a
@santimintegui @matiasrb97
change segment type into filter
fbb9e2f
@santimintegui @matiasrb97
Add test to check if cmcd data from manifest is correct
dbb2e31
@santimintegui @matiasrb97
delete comments
32b7fa3
@matiasrb97
Update tests with latest changes for cmcd reporting data
d8d58aa
@santimintegui @matiasrb97
add test for getQueryParameter
067ecac
@santiagomintegui @matiasrb97
delete comment in streaming.models.CmcdModel.js
069ddef
@santimintegui @matiasrb97
add test to get correctly cmcd data from getHeadersParameters()
8cd3ff6
@matiasrb97
Adding tests for ClientDataReportingModel
7f7324a
@santimintegui @matiasrb97
add test to get correctly cmcd data from getHeadersParameters()
426a2dd
@santimintegui @matiasrb97
add funcionality to service description controller mock
8728319
@matiasrb97
reset serviceDescriptionControllerMock after tests run
cb61882
@matiasrb97
More tests added for ClientDataReportingModel
f2fa528
@matiasrb97
Add tests for applyCMCDParameters setting
85c7164
@matiasrb97
Add includeInRequests attribute to player settings
969d30f
@N1Knight @matiasrb97
add logger
3d18450
@N1Knight @matiasrb97
improve edge cases checks
a7481dc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@N1Knight N1Knight Awaiting requested review from N1Knight

@arrivillagaTomas arrivillagaTomas Awaiting requested review from arrivillagaTomas

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@matiasrb97 @santimintegui @santiagomintegui @N1Knight