Skip to content
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

Bluetooth media player messaging #77

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

aselafernando
Copy link
Contributor

Created messaging from the Bluetooth media player. This will broadcast the current track details and track position so plugins can make use of this. This will allow writing this data to a vehicle display over CANBUS or serial etc.

@viktorgino
Copy link
Owner

I think it'd be a better idea to a media position and media info changed signal to MediaInterface, so all the other media plugins can use it.
Then MediaManager could handle sending it onto the messaging bus etc

@aselafernando
Copy link
Contributor Author

Yep that makes sense, I have modifed the code so now MediaManager is the interface to the message bus.

MediaManager::mediaPosition quint32 and MediaManager::mediaTrack QVariantMap are the two messages. Just tested this and it worked well. Could probably drop "media" from the id as well.

@aselafernando
Copy link
Contributor Author

I changed the messages to MediaInput::position quint32 and MediaInput::track QVariantMap to be more consistent with the existing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants