Merge pull request #116 from Serafim-End/fix_user_storage

update isinstance of user_storage / multiple buttons per card

Author: treethought

flask_assistant/__init__.py

@@ -26,7 +26,10 @@
     profile
 )
 
-from flask_assistant.response import ask, tell, event, build_item, permission, sign_in
+from flask_assistant.response import (
+    ask, tell, event, build_item, permission, sign_in, build_button
+)
+
 from flask_assistant.manager import Context
 
 import flask_assistant.utils

flask_assistant/response.py

@@ -140,13 +140,26 @@ def card(
         subtitle=None,
         link=None,
         link_title=None,
+        buttons=None,
     ):
+        """
+        :param: link and :param: link_title supports only one button per card
+        in future versions should be deleted
+        and from now deprecated
+
+        :param link:
+        :param link_title:
+        :return:
+        """
 
         card_payload = {"title": title, "subtitle": subtitle, "formattedText": text}
 
-        if link and link_title:
-            btn_payload = [{"title": link_title, "openUriAction": {"uri": link}}]
-            card_payload["buttons"] = btn_payload
+        if buttons:
+            card_payload["buttons"] = buttons
+        elif link and link_title:
+            logger.info('use button parameter instead of link and link_title')
+            buttons = [build_button(title=link_title, link=link)]
+            card_payload["buttons"] = buttons
 
         if img_url:
             img_payload = {"imageUri": img_url, "accessibilityText": img_alt or img_url}
@@ -241,6 +254,13 @@ def add_media(self, url, name, description=None, icon_url=None, icon_alt=None):
         return self
 
 
+def build_button(title, link):
+    return {
+        "title": title,
+        "openUriAction": {"uri": link}
+    }
+
+
 def build_item(
     title, key=None, synonyms=None, description=None, img_url=None, alt_text=None
 ):