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

Added the shared now playing implementation #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vanlooverenkoen
Copy link

No description provided.

Copy link

@bensalcie bensalcie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes work for me, this pr should be approved.

@oguzhnatly
Copy link
Owner

Thank you for your contribution @vanlooverenkoen!
I'm testing on real devices and cars as well. I will be approving and merging this PR as soon as possible.

@oguzhnatly oguzhnatly linked an issue Nov 9, 2022 that may be closed by this pull request
@oguzhnatly oguzhnatly added new-feature New feature or request in testing This issue has been testing labels Nov 9, 2022
@ghost
Copy link

ghost commented Nov 10, 2022

Works fine, but is it possible in the current state to display a badge in the corner to open this tab like on Spotify?

@ghost
Copy link

ghost commented Nov 22, 2022

@oguzhnatly @vanlooverenkoen what is the status of the Now Playing feature? when it will be available?

@vanlooverenkoen
Copy link
Author

@Subhangi3 the status from my side is that we are using this in production since december.

@daniel-naegele
Copy link

@Subhangi3 the status from my side is that we are using this in production since december.

Same, we are also using it in addition with #25. Everything seems to be working fine.

@maendamedia
Copy link

maendamedia commented Nov 15, 2023

@vanlooverenkoen Can you tell me how you did that please?
I need it urgent.

@vanlooverenkoen
Copy link
Author

I just used this fork

@maendamedia
Copy link

I just used this fork

Sure that part I understand :D

But I keep getting: Thread 1: "Application does not implement CarPlay template application lifecycle methods in its scene delegate."

Any idea how to fix that?

@maendamedia
Copy link

maendamedia commented Nov 15, 2023

@vanlooverenkoen, could you please guide me in the right direction for the initialization and correct display on CarPlay?

I am using the radio_player package for the music part.

Mainly, I am receiving the following notifications:

vbnet
Copy code
AddInstanceForFactory: No factory registered for id <CFUUID 0x6000002afa40> F8BB1C28-BAE8-11D6-9C31-00039315CD46

MSVEntitlementUtilities - Process Runner PID[34.048] - Group: - Entitlement: com.apple.mediaremote.external-artwork-validation - Entitled: 0 - Error:

93.955 HALC_ProxyIOContext.cpp:1.329 HALC_ProxyIOContext::IOWorkLoop: skipping cycle due to overload

HALB_MachPort.cpp:174 HALB_MachPort::DestroyPort: not going to destroy anything this way
Additionally, I don't see any display on the CarPlay screen.

I hope you can point me in the right direction.

@maendamedia
Copy link

Can someone please help me with this how to show now playing on screen?

@AlexMoumou
Copy link

Hello is this dead for some reason? Id like to use it for a freelance project.

@im-sacha-cohen
Copy link

im-sacha-cohen commented Aug 29, 2024

Hi there, any news ? @oguzhnatly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in testing This issue has been testing new-feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New Feature: Now Playing
7 participants