Skip to content

Allow getting the device info from PillarboxCastPlayer#965

Merged
StaehliJ merged 1 commit into928-custom-castplayerfrom
937-get-device-info
Apr 2, 2025
Merged

Allow getting the device info from PillarboxCastPlayer#965
StaehliJ merged 1 commit into928-custom-castplayerfrom
937-get-device-info

Conversation

@MGaetan89
Copy link
Contributor

Pull request

Description

This PR adds the necessary logic for the getDeviceInfo() to function as expected. The implementation is based on what is done in Media3 1.6.0.

Changes made

  • Use MediaRouter2 from Android API 30+ to observe controller changes.
  • Set the deviceInfo field of the Player State object.

Checklist

  • APIs have been properly documented (if relevant).
  • The documentation has been updated (if relevant).
  • New unit tests have been written (if relevant).
  • The demo has been updated (if relevant).

@MGaetan89 MGaetan89 self-assigned this Mar 31, 2025
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Pillarbox Mar 31, 2025
@MGaetan89 MGaetan89 linked an issue Mar 31, 2025 that may be closed by this pull request
@github-actions
Copy link

Code Coverage

Overall Project 49.53% -0.66% 🟢
Files changed 4.05% 🟢

Module Coverage
:pillarbox-cast 7.58% -11.53% 🟢
Files
Module File Coverage
:pillarbox-cast PillarboxCastPlayer.kt 4.05% -14.92% 🟢
PillarboxCastPlayerBuilder.kt 0% -8.48% 🟢

@MGaetan89 MGaetan89 marked this pull request as ready for review March 31, 2025 09:17
@MGaetan89 MGaetan89 requested a review from StaehliJ March 31, 2025 09:17
@MGaetan89 MGaetan89 force-pushed the 937-get-device-info branch from ac9e0e1 to 056228b Compare April 2, 2025 08:25
@StaehliJ StaehliJ merged commit afb8249 into 928-custom-castplayer Apr 2, 2025
7 of 10 checks passed
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in Pillarbox Apr 2, 2025
@StaehliJ StaehliJ deleted the 937-get-device-info branch April 2, 2025 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Get device info

2 participants