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

Feat: add audioitem for echo spot #255

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

Conversation

senufo
Copy link

@senufo senufo commented Aug 25, 2019

Hi,
I added the metadata parameter in audioitem as described here:
https://developer.amazon.com/fr/docs/custom-skills/audioplayer-interface-reference.html#play
With the metadata we add the support for the screens alexa devices. I tested on Echo Spot.

WIth this an object providing metadata about the audio to be displayed on the Alexa-enabled device with a screen. Note that this metadata displays only on the devices. It is not shown in the Alexa app.

Regards

"""Adds stream to the queue. Does not impact the currently playing stream."""
directive = self._play_directive('ENQUEUE')
audio_item = self._audio_item(stream_url=stream_url,
offset=offset,
push_buffer=False,
opaque_token=opaque_token)
opaque_token=opaque_token
Copy link

Choose a reason for hiding this comment

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

you're missing a comma here

@samarulrajt
Copy link

Hi,
I tried the audio samples from samples folder flask-ask/samples/audio/simple_demo/. When testing this inn alexa test console, it throws an error.

image

What could be the problem?

Thanks,
Sam

@phirestalker
Copy link

I am using ask sonic. They use this library, and I was looking to add this functionality there. I thought it was a simple fix, because the Alexa device does receive a response with metadata and the URL for the album cover. However, the height and width properties are set to 0. I guess this causes it not to be displayed? Is that something that an issue should be opened for, or does your code not send anything that would cause the URL to fill in?

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.

4 participants