Merge pull request #85 from AdaGold/ks_update_cli

Update CLI for AirPlay & changed endpoint responses

Author: kelsey-steven-ada

cli/main.py

@@ -13,8 +13,8 @@
         "10": "Quit"
         }
 
-def list_options():
 
+def list_options():
     for number, feature in OPTIONS.items():
         print(f"{number}. {feature}")
 
@@ -29,14 +29,15 @@ def make_choice():
 
     return choice
 
+
 def get_task_from_user(msg = "Input the id of the task you would like to work with: "):
     task = None
     tasks = task_list.list_tasks()
     if not tasks:
         task_list.print_stars("This option is not possible because there are no tasks.")
         return task
     count = 0
-    help_count = 3 #number of tries before offering assistance
+    help_count = 3 # number of tries before offering assistance
     while not task:
         id = input(msg)
         task = task_list.get_task(id)
@@ -49,6 +50,7 @@ def get_task_from_user(msg = "Input the id of the task you would like to work wi
 
     return task
 
+
 def print_task(task):
     print_single_row_of_stars()
     print("title: ", task["title"])
@@ -57,45 +59,56 @@ def print_task(task):
     print("id: ", task["id"])
     print_single_row_of_stars()
 
+
 def print_all_tasks():
     tasks = task_list.list_tasks()
+
     print("\nTasks:")
     if not tasks:
         print_surround_stars("No tasks")
     else:
         for task in tasks:
             print_task(task)
+
     print_single_row_of_stars()
 
+
 def print_surround_stars(sentence):
     print_single_row_of_stars()
     print(sentence)
     print_single_row_of_stars()
 
+
 def print_single_row_of_stars():
     print("\n**************************\n")
 
+
 def create_task():
     print("Great! Let's create a new task.")
     title=input("What is the title of your task? ")
     description=input("What is the description of your task? ")
     response = task_list.create_task(title, description)
     print_task(response)
 
+
 def view_task():
     task = get_task_from_user("Input the id of the task you would like to select ")
     if task: 
         print("\nSelected Task:")
         print_task(task)
 
+
 def edit_task():
     task = get_task_from_user()
     if task:
         title=input("What is the new title of your task? ")
         description=input("What is the new description of your task? ")
-        response = task_list.update_task(task["id"], title, description)
+        task_list.update_task(task["id"], title, description)
+
         print("\nUpdated Task:")
-        print_task(response)
+        updated_task = task_list.get_task(task["id"])
+        print_task(updated_task)
+
 
 def delete_task_ui():
     task = get_task_from_user("Input the id of the task you would like to delete: ")
@@ -104,26 +117,30 @@ def delete_task_ui():
         print("\nTask has been deleted.")
         print_all_tasks()
 
+
 def change_task_complete_status(status):
     status_text = "complete"
     if not status:
         status_text = "incomplete"
     task = get_task_from_user(f"Input the id of the task you would like to mark {status_text}: ")
     if task:
         if status:
-            response = task_list.mark_complete(task["id"])
+            task_list.mark_complete(task["id"])
         else:
-            response = task_list.mark_incomplete(task["id"])
+            task_list.mark_incomplete(task["id"])
+
         print(f"\nTask marked {status_text}:")
-        print_task(response)
+        updated_task = task_list.get_task(task["id"])
+        print_task(updated_task)
+
 
 def delete_all_tasks():
     for task in task_list.list_tasks():
         task_list.delete_task(task["id"])
         print_surround_stars("Deleted all tasks.")
 
+
 def run_cli():
-    
     play = True
     while play:
 

cli/task_list.py

@@ -1,8 +1,8 @@
 import requests
 
-url = "http://localhost:5000"
+url = "http://127.0.0.1:5000"
 
-def parse_response(response):
+def parse_task_from_response(response):
     if response.status_code >= 400:
         return None
 
@@ -15,18 +15,21 @@ def create_task(title, description, completed_at=None):
         "completed_at": completed_at
     }
     response = requests.post(url+"/tasks",json=query_params)
-    return parse_response(response)
+    return parse_task_from_response(response)
+
 
 def list_tasks():
     response = requests.get(url+"/tasks")
     return response.json()
 
+
 def get_task(id):
     response = requests.get(url+f"/tasks/{id}")
     if response.status_code != 200:
         return None
 
-    return parse_response(response)
+    return parse_task_from_response(response)
+
 
 def update_task(id,title,description):
 
@@ -40,20 +43,20 @@ def update_task(id,title,description):
         json=query_params
         )
 
-    return parse_response(response)
+    return response
+
 
 def delete_task(id):
     response = requests.delete(url+f"/tasks/{id}")
-    return response.json()
+    return response
+
 
 def mark_complete(id):
     response = requests.patch(url+f"/tasks/{id}/mark_complete")
-    return parse_response(response)
+    return response
+
 
 def mark_incomplete(id):
     response = requests.patch(url+f"/tasks/{id}/mark_incomplete")
-    return parse_response(response)
-
+    return response
 
-
-