Spotify Play?

mantha

if two or more people came to the same conclusion, its an indication that its not an to bad idea :-)
independent from this post i just had setup my mirror exactly the way @yawns and @richland007 described to you.

I enhanced MMM-NowPlayingOnSpotify and MMM-AssistantMk2 to make them capable to control Spotify. It´s still in development but if you like you can have a look on my git forks.

MMM-NowPlayingOnSpotify
MMM-AssistantMk2

you need to redo your authorization (Step 2) because we require now also the authorization user-modify-playback-state and you should add “deviceName” into your config-file.

Im looking forward to your feedback dear alpha-tester ;-)

F17MC

@mantha Hi! How can I install this feature?
I want launch Spotify with my MagicMirror. I have installed NowPlayingOnSpotify, AssistantMK2 and Raspotify. All works fine.
Now, I can do the next: If I said to AssistantMK2 “SmartMirror, play Spotify on Chromecast Audio”, it launch raspotify and select my ChromeCast Audio as output. Do you think that it is possible?

Sorry for my English.

ejay-ibm

@mantha Thank you for working on this !

I have installed you modules.

Unfortunately I can’t make the player respond to my play / stop command .
the player well respond when I select a song from my phone and send it to raspotify , the module is well updated with cover and paying status .

But voice command just doesn’t work .

Anything particular I should say ?
Maybe your developement is just done 100% done yet.

Thank you

Ejay

mantha

@ejay-ibm
the module is listening to an open.spotify.com answer from Google assistant.
It seems like the assistant is not aware of the preconfigured music provider and you need to point him in the right direction.

“search for xxx on Spotify” works in my case. I’m almost sure that in the past also “play xxx from spotify” had worked, but now I got no more an open.spotify.com link backs. Would be interesting how it works for you.

FYI - I identified that the Device parameter does not work proper in the current github version. need to work on this…

ejay-ibm

Ok I see the answer coming to the screen but it look like it’s not passed to the module.

"Micheal Jackson on spotify" ((open.spotify.com -- https://open.spotify.com/artist/3fMbdgg4jU18AjLCKBhRSm))\nMichael Jackson wasn't merely the biggest pop ....

Let me know if you need me to run some test or report logs to help you in debug.

Ejay

mantha

@ejay-ibm
in a first step you could check the console log for “[AMK2] openSpotify found:” and if the event PLAY_SPOTIFY was fired…

ejay-ibm

@mantha none of both found .
could you point me to the line in your code sending this to the console log please ?
I see in MK2 js code

if (payload.foundOpenSpotify) {
		this.sendNotification("PLAY_SPOTIFY", {
			url: payload.foundOpenSpotify
		})
    };

I can say that the notification is not sent, so no payload.foundOpenSpotify occur

I can see as well in node_helper.js

if (openSpotify) {
          console.error("[AMK2] openSpotify found:", openSpotify[1])
          foundOpenSpotify = openSpotify[1]
        }
      })

Why in console error instead of console log ?
( Sorry i’m not a dev might be a silly question and I guess the result would be the same if log or error it should be sent to the console… )

ejay-ibm

As well I think the regex is not correct here : node_helper.js line 290

var re = new RegExp("\\(.open\\.spotify\\.com – (https:\\/\\/open\\.spotify\\.com[^ ]*).\\)", "gm")

should be :

var re = new RegExp("\(open\.spotify\.com – (https:\/\/open\.spotify\.com[^ ]*\/\)", "gm")

open\.spotify\.com

is the correct regex to match with :
open.spotify.com
and

https:\/\/open\.spotify\.com[^ ]*\/

is the correct regex to match with :
https://open.spotify.com/fallback1234

I may be wrong in my interpretation.

mantha

@ejay-ibm said in Spotify Play?:

Why in console error instead of console log ?
( Sorry i’m not a dev might be a silly question and I guess the result would be the same if log or error it should be sent to the console… )

haha… there is a simple answer. I want them to be red to find them more easy. In a final version it should be console log…

Regarding the RegEx: I just tested them in german. I’m for sure not an RegEx pro (that’s a kind of black magic for me) ;-)
So if you have a better Regex expression feel free to change it.

The additional escape characters was necessary to wrap the RegEx into the java code.
Are you getting it to work without these escapes?

Proposal:
based on your post it seems to me, like the English version respond with

open.spotify.com -- https...

instead of

open.spotify.com - https...

So you might need to reflect this in the RegEx

var re = new RegExp("\\(.open\\.spotify\\.com –+ (https:\\/\\/open\\.spotify\\.com[^ ]*).\\)", "gm")

ejay-ibm

@mantha
Sorry typo it well respond with

open.spotify.com - https...

I have try :

 var re = new RegExp("https:\/\/open\.spotify\.com\/([^ ]*)", "gm")
        var openSpotify = re.exec(str)
        if (openSpotify) {
          console.error("[AMK2] openSpotify found:", openSpotify[1])
          foundOpenSpotify = openSpotify[1]

And I’m getting in console log :

received a module notification: PLAY_SPOTIFY from sender: MMM-AssistantMk2

but

var re = new RegExp("\\(.open\\.spotify\\.com – (https:\\/\\/open\\.spotify\\.com[^ ]*).\\)", "gm")

I’m not getting anything.

What is strange is that in both case :
console.error(“[AMK2] openSpotify found:”, openSpotify[1])

Doesn’t throw anything…
but as well
console.log(“[AMK2] video found:”, youtubeVideo[1])
Doesn’t throw anything either even when it find and play a video. so i’ll not worry for not about the notif logs

So, for Now, I consider it’s working as PLAY_SPOTIFY notification is fired.

What would be the next step now?

Ejay