forslund / spotify-skill

Mycroft Skill to control spotify using the Spotify Connect API
Apache License 2.0
72 stars 38 forks source link

Better feedback #62

Closed g3rb3n closed 5 years ago

g3rb3n commented 5 years ago

Changed constructs like:

def shuffle_on(self):
    if self.spotify:
        self.spotify.shuffle(True)

To

def shuffle_on(self):
    if not self.spotify:
        self.speak_dialog('NotConnected')
        return
    self.spotify.shuffle(True)
forslund commented 5 years ago

Can you re-open this to the 18.08 branch.

g3rb3n commented 5 years ago

Sure no problem.