-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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.
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
Add reason for PlaybackActiveTrackChanged event #2426
Comments
its an FR alright. ur looking at emitPlaybackTrackChangedEvents on android and wtv for apple. u can add a field in the bundle denoting the reason if ud like. |
Don't know what is an FR. I can add the reason myself but the android code is hard to understand and navigate. 🙂 |
fr = feature request
and the feature is something u want, not me. the best way moving forward is
diy or hire someone.
…On Mon, Jan 6, 2025, 9:50 AM Aman Rohilla ***@***.***> wrote:
Don't know what is an FR.
I can add the reason myself but the android code is hard to understand and
navigate. 🙂
—
Reply to this email directly, view it on GitHub
<#2426 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AZMOVVSIM7U5VM3DR5U2TLD2JK67VAVCNFSM6AAAAABUPCLAO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNZTGYYTENZSGQ>
.
You are receiving this because you commented.Message ID:
***@***.***
com>
|
Ok, I can do it myself. Just wanted to bring this issue to attention. If some maintainer works on it, then everybody can benifit from it. |
We should be able to know that the current track got changed because of it's playback has ended or some another reason (like some other track got inserted).
For this, we can either add a reason for PlaybackActiveTrackChanged event or introduce another event like PlaybackFinished.
The text was updated successfully, but these errors were encountered: