Skip to content

feat(srg-analytics): remove Pillarbox dependency#306

Draft
amtins wants to merge 1 commit intomainfrom
feat/srg-analytics-remove-pillarbox-dependency
Draft

feat(srg-analytics): remove Pillarbox dependency#306
amtins wants to merge 1 commit intomainfrom
feat/srg-analytics-remove-pillarbox-dependency

Conversation

@amtins
Copy link
Member

@amtins amtins commented Mar 14, 2025

Description

Eliminates dependency on Pillarbox and therefore relies solely on the media element for data collection. It also facilitates the use of any other type of web player.

Changes made

  • SRGAnalytics
    • replace Pillarbox references with media element
    • add functions to handle live streams
    • update unit tests
  • SrgSsr
    • add function set the playback data
    • add loadstart handler to update the playback data
  • adapt unit tests

@github-actions
Copy link

github-actions bot commented Mar 14, 2025

PR Preview Action v1.6.0

🚀 View preview at
https://SRGSSR.github.io/pillarbox-web/pr-preview/pr-306/

Built to branch gh-pages at 2025-03-14 18:05 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@github-actions
Copy link

github-actions bot commented Mar 14, 2025

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements
97.03% (-1.08% 🔻)
818/843
🟢 Branches
92.43% (-0.59% 🔻)
391/423
🟢 Functions
96.98% (-1.71% 🔻)
225/232
🟢 Lines
97.23% (-1.27% 🔻)
773/795
Show files with reduced coverage 🔻
St.
File Statements Branches Functions Lines
🟢
... / SRGAnalytics.js
94.47% (-3.07% 🔻)
88.55% (-1.77% 🔻)
94.55% (-5.45% 🔻)
94.74% (-3.59% 🔻)
🟢
... / srgssr.js
98.79% (-0.59% 🔻)
98.96%
98% (-2% 🔻)
98.61% (-0.67% 🔻)

Test suite run success

248 tests passing in 10 suites.

Report generated by 🧪jest coverage report action from 9b573b4

Eliminates dependency on Pillarbox and therefore relies solely on the media
element for data collection. It also facilitates the use of any other type of
web player.

- SRGAnalytics
  - replace Pillarbox references with media element
  - add functions to handle live streams
  - update unit tests
- SrgSsr
  - add function set the playback data
  - add loadstart handler to update the playback data
- adapt unit tests
@amtins amtins force-pushed the feat/srg-analytics-remove-pillarbox-dependency branch from e10d452 to 9b573b4 Compare March 14, 2025 18:04
@defagos defagos moved this to 📋 Backlog in Pillarbox Mar 17, 2025
@amtins amtins moved this from 📋 Backlog to 🚧 In Progress in Pillarbox Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🚧 In Progress

Development

Successfully merging this pull request may close these issues.

1 participant

Comments