Remove displayError, use displaySuccess

presleyp · presleyp · commit c2f6cce93863 · 2022-04-26T02:19:47.000Z
diff --git a/site/src/components/GlobalSnackbar/utils.ts b/site/src/components/GlobalSnackbar/utils.ts
@@ -60,7 +60,3 @@ export const displayMsg = (msg: string, additionalMsg?: string): void => {
 export const displaySuccess = (msg: string, additionalMsg?: string): void => {
   dispatchNotificationEvent(MsgType.Success, msg, additionalMsg ? [additionalMsg] : undefined)
 }
-
-export const displayError = (msg: string, additionalMsg?: string): void => {
-  dispatchNotificationEvent(MsgType.Error, msg, additionalMsg ? [additionalMsg] : undefined)
-}
diff --git a/site/src/xServices/users/usersXService.ts b/site/src/xServices/users/usersXService.ts
@@ -2,10 +2,9 @@ import { assign, createMachine } from "xstate"
 import * as API from "../../api"
 import * as Types from "../../api/types"
 import * as TypesGen from "../../api/typesGenerated"
-import { displayError } from "../../components/GlobalSnackbar/utils"
+import { displaySuccess } from "../../components/GlobalSnackbar/utils"
 
-const Language = {
-  createUserError: "Unable to create user",
+export const Language = {
   createUserSuccess: "Successfully created user",
 }
 
@@ -74,7 +73,7 @@ export const usersMachine = createMachine(
           },
           onError: {
             target: "idle",
-            actions: ["assignCreateUserError", "displayCreateUserError"],
+            actions: ["assignCreateUserError"],
           },
         },
         tags: "loading",
@@ -110,11 +109,8 @@ export const usersMachine = createMachine(
         ...context,
         createUserError: undefined,
       })),
-      displayCreateUserError: (_, event) => {
-        displayError(Language.createUserError)
-      },
       displayCreateUserSuccess: () => {
-        displayError(Language.createUserSuccess)
+        displaySuccess(Language.createUserSuccess)
       },
     },
   },

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,3 @@ export const displayMsg = (msg: string, additionalMsg?: string): void => {`
`60`	`60`	`export const displaySuccess = (msg: string, additionalMsg?: string): void => {`
`61`	`61`	`dispatchNotificationEvent(MsgType.Success, msg, additionalMsg ? [additionalMsg] : undefined)`
`62`	`62`	`}`
`63`		`-`
`64`		`-export const displayError = (msg: string, additionalMsg?: string): void => {`
`65`		`- dispatchNotificationEvent(MsgType.Error, msg, additionalMsg ? [additionalMsg] : undefined)`
`66`		`-}`