Nathandegraaf error snack (#232)

* Graph polishing * Revert "Graph polishing" This reverts commit 65cfb1b. * Adds Error Snackbar on Server errors * fix lint
hammer-io · Apr 17, 2018 · a9e402b · a9e402b
1 parent 92f4033
commit a9e402b
Show file tree

Hide file tree

Showing 12 changed files with 199 additions and 76 deletions.
diff --git a/src/actions/error.js b/src/actions/error.js
@@ -0,0 +1,11 @@
+/* eslint import/prefer-default-export: 0 */
+import actionCreator from '../utils/actionCreator'
+import * as Constants from '../constants'
+
+export function setError(errorMessage) {
+  return actionCreator(Constants.SET_ERROR, { errorMessage })
+}
+
+export function hideError() {
+  return actionCreator(Constants.HIDE_ERROR)
+}
diff --git a/src/actions/invite.js b/src/actions/invite.js
@@ -1,6 +1,7 @@
 import FetchClient from './../utils/fetchClient'
 import actionCreator from './../utils/actionCreator'
 import * as Constants from './../constants'
+import logError from '../utils/error'
 
 export function setUserInvites(invites) {
   return actionCreator(Constants.SET_USER_INVITES, { invites })
@@ -16,10 +17,10 @@ export function getUserInvites(token) {
         dispatch(setUserInvites(result))
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }
@@ -37,10 +38,10 @@ function updateInvite(token, inviteId, action) {
         dispatch(setUserInvites([result]))
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }

diff --git a/src/actions/project.js b/src/actions/project.js
@@ -2,6 +2,7 @@ import download from './../vendor/download'
 import FetchClient from './../utils/fetchClient'
 import actionCreator from './../utils/actionCreator'
 import * as Constants from './../constants'
+import logError from '../utils/error'
 
 export function setUserProjects(projects) {
   return actionCreator(Constants.SET_USER_PROJECTS, { projects })
@@ -41,10 +42,10 @@ export function getUserProjects(token) {
         dispatch(setUserProjects(result))
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }
@@ -62,10 +63,10 @@ export function getIssues(token, id, parameters) {
         dispatch(setIssues(result))
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }
@@ -83,10 +84,10 @@ export function getBuildStatuses(token, id, limit) {
         dispatch(setBuildStatuses(result))
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }
@@ -104,10 +105,10 @@ export function getProjectContributors(token, id) {
         dispatch(setProjectContributors(result))
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }
@@ -123,10 +124,10 @@ export function getProjectOwners(token, id) {
         dispatch(setProjectOwners(result))
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }
@@ -142,10 +143,10 @@ export function getProjects(token) {
         dispatch(setProjects(result))
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }
@@ -161,10 +162,10 @@ export function getProject(token, id) {
         dispatch(setProjects([result]))
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }
@@ -190,7 +191,7 @@ export function deleteProject(token, id) {
 }
 
 export function getProjectFiles(token, id) {
-  return async () => {
+  return async (dispatch) => {
     try {
       const fetchClient = new FetchClient()
       fetchClient.setAuthToken(token)
@@ -205,10 +206,10 @@ export function getProjectFiles(token, id) {
         download(result, 'project.zip')
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }
@@ -226,10 +227,10 @@ export function addContributor(token, info) {
         dispatch(setProjectContributors(result))
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }
@@ -247,10 +248,10 @@ export function addOwner(token, info) {
         dispatch(setProjectOwners(result))
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }
@@ -268,10 +269,10 @@ export function removeContributor(token, info) {
         dispatch(setProjectContributors(result))
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }
@@ -289,10 +290,10 @@ export function removeOwner(token, info) {
         dispatch(setProjectOwners(result))
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }
@@ -311,10 +312,10 @@ export function addProject(token, project) {
         dispatch(setProject(result))
         return { result, error }
       }
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     } catch (error) {
-      console.error(error)
+      logError(dispatch, error)
       return { result: null, error }
     }
   }