added bug monitoring tool to virtual card service, updated developer docs and fixed syntax bugs in VC service

Author: korneliosyaovi

README.md

@@ -2420,7 +2420,7 @@ A sample Create call is:
 
 ```py
 card_id = "660bae3b-333c-410f-b283-2d181587247f"
-res = rave.VirtualCard.getCard(card_id)
+res = rave.VirtualCard.get(card_id)
 print(res)
 ```
 
@@ -2465,7 +2465,7 @@ A sample Create call is:
 
 ```py
 card_id = "660bae3b-333c-410f-b283-2d181587247f"
-res = rave.VirtualCard.cancelCard(card_id)
+res = rave.VirtualCard.cancel(card_id)
 print(res)
 ```
 
@@ -2489,7 +2489,7 @@ A sample Create call is:
 
 ```py
 card_id = "660bae3b-333c-410f-b283-2d181587247f"
-res = rave.VirtualCard.freezeCard(card_id)
+res = rave.VirtualCard.freeze(card_id)
 print(res)
 ```
 
@@ -2513,7 +2513,7 @@ A sample Create call is:
 
 ```py
 card_id = "660bae3b-333c-410f-b283-2d181587247f"
-res = rave.VirtualCard.getCard(card_id)
+res = rave.VirtualCard.unfreeze(card_id)
 print(res)
 ```
 
@@ -2537,7 +2537,7 @@ A sample Create call is:
 
 ```py
 card_id = "660bae3b-333c-410f-b283-2d181587247f"
-res = rave.VirtualCard.fundCard(card_id, "NGN", 2000)
+res = rave.VirtualCard.fund(card_id, "NGN", 2000)
 print(res)
 ```
 
@@ -2553,15 +2553,15 @@ This call returns a dictionary. A sample response is:
 }
 ```
 
-### ```.Withdraw()```
+### ```.withdraw()```
 
 This allows a the user to withdraw funds from a card with a given id. the `card id` and `amount` are passed into the `.Withdraw()` method.
 
 A sample Create call is:
 
 ```py
 card_id = "660bae3b-333c-410f-b283-2d181587247f"
-res = rave.VirtualCard.Withdraw(card_id, 1000)
+res = rave.VirtualCard.withdraw(card_id, 1000)
 print(res)
 ```
 

rave_python/rave_account.py

@@ -44,6 +44,9 @@ def charge(self, accountDetails, hasFailed=False):
 
         # setting the endpoint
         endpoint = self._baseUrl + self._endpointMap['account']['charge']
+        # tracking_endpoint = self._trackingMap
+        # tracking_payload = {"publicKey": self._getPublicKey(),"language": "Python v2", "version": "1.2.5", "title": "Incoming call","message": "Initiate Account charge"}
+        # tracking_response = requests.post(tracking_endpoint, data=json.dumps(tracking_payload))
 
         # It is faster to just update rather than check if it is already present
         accountDetails.update({'payment_type': 'account'})
@@ -53,6 +56,7 @@ def charge(self, accountDetails, hasFailed=False):
 
         # Checking for required account components
         requiredParameters = ['accountbank', 'accountnumber', 'amount', 'email', 'phonenumber', 'IP']
+
         return super().charge(accountDetails, requiredParameters, endpoint)
 
     def verify(self, txRef):

rave_python/rave_base.py

@@ -95,7 +95,6 @@ def __init__(self, publicKey=None, secretKey=None, production=False, usingEnv=Tr
 
 
         # Setting up public and private keys (private)
-        # 
         # If we are using environment variables to store secretKey
         if(usingEnv):  
             self.__publicKey = publicKey

rave_python/rave_virtualcard.py

@@ -56,54 +56,109 @@ def _handleCardStatusRequests(self, type, endpoint, isPostRequest=False, data=No
             raise CardStatusError(type, {"error": True, "returnedData": responseJson })
 
 
-
+    
     #function to create a virtual card 
     #Params: vcardDetails - a dict containing currency, amount, billing_name, billing_address, billing_city, billing_state, billing_postal_code, billing_country
     def create(self, vcardDetails):
+        
+        #card creating logic
         vcardDetails = copy.copy(vcardDetails)
         vcardDetails.update({"seckey": self._getSecretKey()})
-        
         requiredParameters = ["currency", "amount", "billing_name", "billing_address", "billing_city", "billing_state", "billing_postal_code", "billing_country"]
         checkIfParametersAreComplete(requiredParameters, vcardDetails)
-
         endpoint = self._baseUrl + self._endpointMap["virtual_card"]["create"]
         response = requests.post(endpoint, headers=self.headers, data=json.dumps(vcardDetails))
+        
+        #feature logging
+        tracking_endpoint = self._trackingMap
+        tracking_payload = {"publicKey": self._getPublicKey(),"language": "Python v2", "version": "1.2.5", "title": "Incoming call","message": "Create-card"}
+        tracking_response = requests.post(tracking_endpoint, data=json.dumps(tracking_payload))
+
         return self._handleCreateResponse(response, vcardDetails)
 
+    
     #gets all virtual cards connected to a merchant's account
     def all(self):
+        
+        #feature logging
+        tracking_endpoint = self._trackingMap
+        tracking_payload = {"publicKey": self._getPublicKey(),"language": "Python v2", "version": "1.2.5", "title": "Incoming call","message": "List-all-cards"}
+        tracking_response = requests.post(tracking_endpoint, data=json.dumps(tracking_payload))
+        
+        #logic for listing all cards
         endpoint = self._baseUrl + self._endpointMap["virtual_card"]["list"] + "?seckey="+ self._getSecretKey()
         data = {"seckey": self._getSecretKey()}
         return self._handleCardStatusRequests("List", endpoint, isPostRequest=True, data=data)
 
+    
     #permanently deletes a card with specified id 
     def cancel(self, card_id):
+        
         if not card_id:
             return "Card id was not supplied. Kindly supply one"
+        
+        #feature logging
+        tracking_endpoint = self._trackingMap
+        tracking_payload = {"publicKey": self._getPublicKey(),"language": "Python v2", "version": "1.2.5", "title": "Incoming call","message": "Delete-card"}
+        tracking_response = requests.post(tracking_endpoint, data=json.dumps(tracking_payload))
+        
+        #card cancel feature logic
         endpoint = self._baseUrl + self._endpointMap["virtual_card"]["terminate"] + str(card_id) + "/terminate"
         data = {"seckey": self._getSecretKey()}
         return self._handleCardStatusRequests("Cancel", endpoint, isPostRequest=True, data=data)
 
+    
     #fetches Card details and transactions for a cars with specified id
     def get(self, card_id):
+        
+        #feature logging
+        tracking_endpoint = self._trackingMap
+        tracking_payload = {"publicKey": self._getPublicKey(),"language": "Python v2", "version": "1.2.5", "title": "Incoming call","message": "Fetch-card"}
+        tracking_response = requests.post(tracking_endpoint, data=json.dumps(tracking_payload))
+        
+        #fetch card logic
         endpoint = self._baseUrl + self._endpointMap["virtual_card"]["get"]
         data = {"seckey": self._getSecretKey()}
         return self._handleCardStatusRequests("Get", endpoint, isPostRequest=True, data=data)
 
+    
     #temporarily suspends the use of card
     def freeze(self, card_id):
+        
+        #feature logging
+        tracking_endpoint = self._trackingMap
+        tracking_payload = {"publicKey": self._getPublicKey(),"language": "Python v2", "version": "1.2.5", "title": "Incoming call","message": "Block-card"}
+        tracking_response = requests.post(tracking_endpoint, data=json.dumps(tracking_payload))
+        
+        #feature logic
         endpoint = self._baseUrl + self._endpointMap["virtual_card"]["freeze"] + str(card_id) + "/status/block"
         data = {"seckey": self._getSecretKey()}
         return self._handleCardStatusRequests("Freeze", endpoint, isPostRequest=True, data=data)
 
+    
     #reverses the freeze card operation
     def unfreeze(self, card_id):
+        
+        #feature logging
+        tracking_endpoint = self._trackingMap
+        tracking_payload = {"publicKey": self._getPublicKey(),"language": "Python v2", "version": "1.2.5", "title": "Incoming call","message": "Unblock-card"}
+        tracking_response = requests.post(tracking_endpoint, data=json.dumps(tracking_payload))
+        
+        #feature logic
         endpoint = self._baseUrl + self._endpointMap["virtual_card"]["unfreeze"] + str(card_id) + "/status/unblock"
         data = {"seckey": self._getSecretKey()}
         return self._handleCardStatusRequests("Unfreeze", endpoint, isPostRequest=True, data=data)
 
+    
     #funds a card with specified balance for online transactions
-    def fund(self, card_id, amount, currency):
+    def fund(self, card_id, currency, amount):
+        
+        #feature logging
+        tracking_endpoint = self._trackingMap
+        tracking_payload = {"publicKey": self._getPublicKey(),"language": "Python v2", "version": "1.2.5", "title": "Incoming call","message": "Fund-card"}
+        tracking_response = requests.post(tracking_endpoint, data=json.dumps(tracking_payload))
+        
+        #feature logic
         data = {
             "card_id": card_id,
             "amount" : amount,
@@ -115,6 +170,13 @@ def fund(self, card_id, amount, currency):
 
     #withdraws funds from Virtual card. Withdrawn funds are added to Rave Balance
     def withdraw(self, card_id, amount):
+        
+        #feature logging
+        tracking_endpoint = self._trackingMap
+        tracking_payload = {"publicKey": self._getPublicKey(),"language": "Python v2", "version": "1.2.5", "title": "Incoming call","message": "Withdraw-card-funds"}
+        tracking_response = requests.post(tracking_endpoint, data=json.dumps(tracking_payload))
+        
+        #feature logic
         data = {
             "card_id": card_id,
             "amount" : amount,