feat: show task name in notification message (#4975)

Author: blegesse-w

src/shared/copy/notifications/categories/tasks.ts

@@ -81,14 +81,19 @@ export const taskCloneFailed = (
   message: `Failed to clone task ${taskName}: ${additionalMessage} `,
 })
 
-export const taskUpdateFailed = (additionalMessage: string): Notification => ({
+export const taskUpdateFailed = (
+  additionalMessage: string,
+  taskName?: string
+): Notification => ({
   ...defaultErrorNotification,
-  message: `Failed to update task: ${additionalMessage}`,
+  message: `Failed to update task ${
+    taskName ? taskName : ''
+  }: ${additionalMessage}`,
 })
 
-export const taskUpdateSuccess = (): Notification => ({
+export const taskUpdateSuccess = (taskName?: string): Notification => ({
   ...defaultSuccessNotification,
-  message: 'Task was updated successfully',
+  message: `Task ${taskName ? taskName : ''} was updated successfully`,
 })
 
 export const taskImportFailed = (errorMessage: string): Notification => ({

src/tasks/actions/thunks.test.ts

@@ -116,7 +116,7 @@ describe('Tasks.Actions.Thunks', () => {
 
     expect(dispatch.mock.calls[2][0].type).toBe('PUBLISH_NOTIFICATION')
     expect(dispatch.mock.calls[2][0].payload.notification).toEqual(
-      taskUpdateSuccess()
+      taskUpdateSuccess(sampleTask.name)
     )
   })
 })

src/tasks/actions/thunks.ts

@@ -231,11 +231,11 @@ export const updateTaskStatus = (task: Task) => async (
 
     dispatch(editTask(normTask))
     dispatch(setCurrentTask(normTask))
-    dispatch(notify(copy.taskUpdateSuccess()))
+    dispatch(notify(copy.taskUpdateSuccess(task.name)))
   } catch (e) {
     console.error(e)
     const message = getErrorMessage(e)
-    dispatch(notify(copy.taskUpdateFailed(message)))
+    dispatch(notify(copy.taskUpdateFailed(message, task.name)))
   }
 }
 
@@ -255,11 +255,11 @@ export const updateTaskName = (name: string, taskID: string) => async (
     )
 
     dispatch(editTask(normTask))
-    dispatch(notify(copy.taskUpdateSuccess()))
+    dispatch(notify(copy.taskUpdateSuccess(name)))
   } catch (e) {
     console.error(e)
     const message = getErrorMessage(e)
-    dispatch(notify(copy.taskUpdateFailed(message)))
+    dispatch(notify(copy.taskUpdateFailed(message, name)))
   }
 }