search

artsy / elderfield

The Artsy Alexa (Echo) skill.
MIT License
33 stars 8 forks source link

Added cards for artists and shows. #50

Closed dblock closed 7 years ago

dblock commented 7 years ago

I added cards to the Alexa app. These look like so:

screen shot 2017-01-06 at 3 57 44 pm

For artists, this is straightforward.

Unfortunately there's no way to add multiple cards, so I changed the shows skill to return only one "recommended" show. That is also easier to consume IMO. You get one piece of info with (this is new) a description and you get a card with more information such as the location of the gallery when available.

ashkan18 commented 7 years ago

Makes sense, make we should randomize shows response in first 5 results so we get (possibly) different show everytime we ask for a recommended show.

dblock commented 7 years ago

@ashkan18 It's randomized via _sample, but actually a better idea is to let the API randomize them, so I am now requesting one show with sample=1 in https://github.com/artsy/elderfield/pull/50/commits/ded8c4e451a29105be06f8133636de0e313e302e.

ashkan18 commented 7 years ago

nice! 💯