Skip to content

Make sure the playback speed is in range ]0,1]#995

Merged
MGaetan89 merged 4 commits into928-custom-castplayerfrom
fix-playback-rate-when-paused
Apr 23, 2025
Merged

Make sure the playback speed is in range ]0,1]#995
MGaetan89 merged 4 commits into928-custom-castplayerfrom
fix-playback-rate-when-paused

Conversation

@StaehliJ
Copy link
Contributor

Pull request

Description

There is an issue with RemoteClient playback rate that can be equal to zero when it is in pause. It can be negative too.

Changes made

  • Ensure the playback rate is valid.

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-project-automation github-project-automation bot moved this to 📋 Backlog in Pillarbox Apr 17, 2025
@StaehliJ StaehliJ requested a review from MGaetan89 April 17, 2025 08:57
@github-actions
Copy link

github-actions bot commented Apr 17, 2025

Code Coverage

Overall Project 48.91% 🟢
Files changed 100% 🟢

Module Coverage
:pillarbox-cast 15.1% 🟢
Files
Module File Coverage
:pillarbox-cast RemoteMediaClient.kt 91.67% 🟢

@defagos defagos changed the title Make sur the playback speed is in range ]0,1] Make sure the playback speed is in range ]0,1] Apr 17, 2025
@MGaetan89 MGaetan89 moved this from 📋 Backlog to 🍿 Code Review in Pillarbox Apr 17, 2025
@MGaetan89 MGaetan89 merged commit 1a37efe into 928-custom-castplayer Apr 23, 2025
9 of 10 checks passed
@MGaetan89 MGaetan89 deleted the fix-playback-rate-when-paused branch April 23, 2025 07:43
@github-project-automation github-project-automation bot moved this from 🍿 Code Review to ✅ Done in Pillarbox Apr 23, 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.

2 participants