Skip to content

967 better handle cast request#1010

Merged
StaehliJ merged 12 commits into928-custom-castplayerfrom
967-better-handle-cast-request
May 6, 2025
Merged

967 better handle cast request#1010
StaehliJ merged 12 commits into928-custom-castplayerfrom
967-better-handle-cast-request

Conversation

@StaehliJ
Copy link
Contributor

@StaehliJ StaehliJ commented May 1, 2025

Pull request

Description

The goal of this PR is to avoid the receiver bad behavior when using commands that make it loading something. For exemple by clicking a lot on skip to next or skip to previous button.

Changes made

  • When RemoteMediaClient is in STATE_LOADING some commands are not available to avoid the receiver to fail.

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).

@github-actions
Copy link

github-actions bot commented May 2, 2025

Code Coverage

Overall Project 49.4% -0.07% 🟢
Files changed 70.59% 🟢

Module Coverage
:pillarbox-core-business-cast 85.16% 🟢
:pillarbox-cast 20.28% -0.85% 🟢
Files
Module File Coverage
:pillarbox-core-business-cast SRGMediaItemConverter.kt 87.9% 🟢
:pillarbox-cast RemoteMediaClient.kt 87.55% -1.75% 🟢
PillarboxCastPlayer.kt 0.55% -0.98% 🟢

@defagos defagos linked an issue May 2, 2025 that may be closed by this pull request
# Conflicts:
#	pillarbox-cast/src/main/java/ch/srgssr/pillarbox/cast/PillarboxCastPlayer.kt
@StaehliJ StaehliJ merged commit a423e3f into 928-custom-castplayer May 6, 2025
10 checks passed
@StaehliJ StaehliJ deleted the 967-better-handle-cast-request branch May 6, 2025 13:54
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in Pillarbox May 6, 2025
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.

Prevent crash of the receiver when handling too many commands

2 participants