Merge pull request #47 from microsoft/jamesqa

Latest styling updates

Author: andrewldesousa

app.py

@@ -885,7 +885,7 @@ async def generate_section_content():
         template = await generate_section_content(request_json)
         return jsonify(template), 200
     except Exception as e:
-        logging.exception("Exception in /history/clear_messages")
+        logging.exception("Exception in /section/generate")
         return jsonify({"error": str(e)}), 500
 
 @bp.route("/document/<filepath>")
@@ -894,7 +894,7 @@ async def get_document(filepath):
         document = retrieve_document(filepath)
         return jsonify(document), 200
     except Exception as e:
-        logging.exception("Exception in /history/clear_messages")
+        logging.exception("Exception in /document/<filepath>")
         return jsonify({"error": str(e)}), 500
 
 async def generate_title(conversation_messages):
@@ -1002,8 +1002,9 @@ async def generate_section_content(request_json):
 
         template = json.loads(response.choices[0].message.content)
         return template
-    except Exception as e:
-        raise e
+    except Exception as ex:
+        logging.exception(ex)
+        return json.loads({"error": str(ex)})
 
 def retrieve_document(filepath):
     try:

frontend/src/api/api.ts

@@ -1,8 +1,21 @@
 import { chatHistorySampleData } from '../constants/chatHistory'
 
-import { ChatMessage, Conversation, ConversationRequest, SectionGenerateRequest, CosmosDBHealth, CosmosDBStatus, UserInfo, ChatType } from './models'
+import {
+  ChatMessage,
+  Conversation,
+  ConversationRequest,
+  SectionGenerateRequest,
+  CosmosDBHealth,
+  CosmosDBStatus,
+  UserInfo,
+  ChatType
+} from './models'
 
-export async function conversationApi(options: ConversationRequest, abortSignal: AbortSignal, chatType: ChatType = ChatType.Browse): Promise<Response> {
+export async function conversationApi(
+  options: ConversationRequest,
+  abortSignal: AbortSignal,
+  chatType: ChatType = ChatType.Browse
+): Promise<Response> {
   const response = await fetch('/conversation', {
     method: 'POST',
     headers: {
@@ -116,7 +129,8 @@ export const historyRead = async (convId: string): Promise<ChatMessage[]> => {
 export const historyGenerate = async (
   options: ConversationRequest,
   abortSignal: AbortSignal,
-  convId?: string, chatType: ChatType = ChatType.Browse
+  convId?: string,
+  chatType: ChatType = ChatType.Browse
 ): Promise<Response> => {
   let body
   if (convId) {
@@ -357,22 +371,20 @@ export const historyMessageFeedback = async (messageId: string, feedback: string
   return response
 }
 
-export const sectionGenerate = async (
-  options: SectionGenerateRequest,
-): Promise<Response> => {
+export const sectionGenerate = async (options: SectionGenerateRequest): Promise<Response> => {
   // set timeout to 10 seconds
   const abortController = new AbortController()
   const abortSignal = abortController.signal
 
   const timeout = setTimeout(() => {
     abortController.abort()
   }, 10000)
-  
+
   let body = JSON.stringify({
     sectionTitle: options.sectionTitle,
-    sectionDescription: options.sectionDescription,
+    sectionDescription: options.sectionDescription
   })
-  
+
   const response = await fetch('/section/generate', {
     method: 'POST',
     headers: {
@@ -388,9 +400,11 @@ export const sectionGenerate = async (
     .catch(_err => {
       clearTimeout(timeout)
       console.error('There was an issue fetching your data.')
-      return new Response()
+      return new Response(
+        JSON.stringify({ section_content: 'There was an issue fetching your data. Please try again.' })
+      )
     })
-  
+
   return response
 }
 
@@ -399,8 +413,12 @@ export const documentRead = async (docId: string): Promise<Response> => {
     method: 'GET',
     headers: { 'Content-Type': 'application/json' }
   })
-    .then(res => { return res })
-    .catch(_err => { throw new Error('There was an issue fetching your data.') })
+    .then(res => {
+      return res
+    })
+    .catch(_err => {
+      throw new Error('There was an issue fetching your data.')
+    })
 
   return response
 }