From 12f88bab525c818ec51c001745506e354a7cdd92 Mon Sep 17 00:00:00 2001
From: Javier San Juan Cervera <Javier.SanJuanCervera@ingrammicro.com>
Date: Fri, 31 May 2019 12:11:47 +0200
Subject: [PATCH] Fixed documentation of FulfillmentAutomation and
 TierConfigAutomation, and added documentation for ActivationTemplateResponse
 and ActivationTileResponse.

---
 connect/models/activation_response.py       | 21 +++++++++++++++++++--
 connect/resources/fulfillment_automation.py | 10 +++++-----
 connect/resources/tier_config_automation.py | 10 +++++-----
 3 files changed, 29 insertions(+), 12 deletions(-)

diff --git a/connect/models/activation_response.py b/connect/models/activation_response.py
index 06b5ff4..cbbfb4c 100644
--- a/connect/models/activation_response.py
+++ b/connect/models/activation_response.py
@@ -7,10 +7,18 @@
 
 
 class ActivationTileResponse(object):
+    """ An instance of this class might the returned by the overriden ``process_request`` method
+    of your :py:class:`connect.resources.FulfillmentAutomation` or
+    :py:class:`connect.resources.TierConfigAutomation` subclass to approve the request being
+    processed, showing a tile with the specified contents.
+
+    :param str markdown: Contents of the tile to be shown in the Vendor Portal, in Markdown format.
+    :rtype: None
+    """
+
     tile = 'Activation succeeded'  # type: str
 
     def __init__(self, markdown=''):
-        # type: (str) -> None
         try:
             self.tile = json.loads(markdown)
         except ValueError:
@@ -18,8 +26,17 @@ def __init__(self, markdown=''):
 
 
 class ActivationTemplateResponse(object):
+    """ An instance of this class might the returned by the overriden ``process_request`` method
+    of your :py:class:`connect.resources.FulfillmentAutomation` or
+    :py:class:`connect.resources.TierConfigAutomation` subclass to approve the request being
+    processed, showing a tile with the specified template id.
+
+    :param str template_id: Id of the template od the tile to be shown in the Vendor Portal.
+        The template must have been defined in the Vendor Portal.
+    :rtype: None
+    """
+
     template_id = None  # type: str
 
     def __init__(self, template_id):
-        # type: (str) -> None
         self.template_id = template_id
diff --git a/connect/resources/fulfillment_automation.py b/connect/resources/fulfillment_automation.py
index 043e798..25dc446 100644
--- a/connect/resources/fulfillment_automation.py
+++ b/connect/resources/fulfillment_automation.py
@@ -18,14 +18,14 @@ class FulfillmentAutomation(AutomationEngine):
     """ This is the automation engine for the Fulfillment API.  If you want to process fulfillment
     requests, subclass this and implement the ``process_request`` method, which receives a
     :py:class:`connect.models.Fulfillment` request as argument and must return an
-    :py:class:`ActivationTemplateResponse` or :py:class:`ActivationTileResponse` object in case
-    the request has to be approved.
+    :py:class:`connect.models.ActivationTemplateResponse` or
+    :py:class:`connect.models.ActivationTileResponse` object in case the request has to be approved.
 
     In other case, you must raise one of these exceptions:
 
-    - :py:class:`connect.models.InquireRequest`: Inquire for more information.
-    - :py:class:`connect.models.FailRequest`: Causes the request to fail.
-    - :py:class:`connect.models.SkipRequest`: Skips processing the request.
+    - :py:class:`connect.exceptions.InquireRequest`: Inquire for more information.
+    - :py:class:`connect.exceptions.FailRequest`: Causes the request to fail.
+    - :py:class:`connect.exceptions.SkipRequest`: Skips processing the request.
 
     Create an instance of your subclass and call its ``process`` method to begin processing.
 
diff --git a/connect/resources/tier_config_automation.py b/connect/resources/tier_config_automation.py
index fa90ad9..b75adc8 100644
--- a/connect/resources/tier_config_automation.py
+++ b/connect/resources/tier_config_automation.py
@@ -16,14 +16,14 @@ class TierConfigAutomation(AutomationEngine):
     """ This is the automation engine for the Tier Config Request API.  If you want to process
     Tier Config requests, subclass this and implement the ``process_request`` method,
     which receives a :py:class:`connect.models.TierConfigRequest` request as argument and must
-    return an :py:class:`ActivationTemplateResponse` or :py:class:`ActivationTileResponse` object
-    in case the request has to be approved.
+    return an :py:class:`connect.models.ActivationTemplateResponse` or
+    :py:class:`connect.models.ActivationTileResponse` object in case the request has to be approved.
 
     In other case, you must raise one of these exceptions:
 
-    - :py:class:`connect.models.InquireRequest`: Inquire for more information.
-    - :py:class:`connect.models.FailRequest`: Causes the request to fail.
-    - :py:class:`connect.models.SkipRequest`: Skips processing the request.
+    - :py:class:`connect.exceptions.InquireRequest`: Inquire for more information.
+    - :py:class:`connect.exceptions.FailRequest`: Causes the request to fail.
+    - :py:class:`connect.exceptions.SkipRequest`: Skips processing the request.
 
     Create an instance of your subclass and call its ``process`` method to begin processing.