Merge pull request #241 from coderoad/fix/error-messages

ShMcK · web-flow · commit 18f3cf661138 · 2020-04-10T21:34:38.000-07:00
add basic error message for tutorial loading
diff --git a/web-app/src/containers/SelectTutorial/LoadTutorialSummary.tsx b/web-app/src/containers/SelectTutorial/LoadTutorialSummary.tsx
@@ -13,9 +13,9 @@ const LoadTutorialSummary = (props: Props) => {
   if (loading) {
     return <Loading text="Loading tutorial summary..." />
   }
-  // TODO: improve error handling
   if (error) {
-    return <div>{JSON.stringify(error)}</div>
+    console.log(error)
+    return <div>Error loading summary</div>
   }
   if (!data) {
     return <div>No data returned for tutorial</div>

Original file line number	Diff line number	Diff line change
`@@ -13,9 +13,9 @@ const LoadTutorialSummary = (props: Props) => {`
`13`	`13`	`if (loading) {`
`14`	`14`	`return <Loading text="Loading tutorial summary..." />`
`15`	`15`	`}`
`16`		`- // TODO: improve error handling`
`17`	`16`	`if (error) {`
`18`		`- return <div>{JSON.stringify(error)}</div>`
	`17`	`+ console.log(error)`
	`18`	`+ return <div>Error loading summary</div>`
`19`	`19`	`}`
`20`	`20`	`if (!data) {`
`21`	`21`	`return <div>No data returned for tutorial</div>`