fix: Error handling for pinned items (#5115)

Author: ChitlangeSahas

src/dashboards/components/dashboard_index/DashboardCard.tsx

@@ -228,10 +228,15 @@ class DashboardCard extends PureComponent<Props> {
     )
   }
 
-  private handleDeleteDashboard = () => {
+  private handleDeleteDashboard = async () => {
     const {id, name, onDeleteDashboard} = this.props
     onDeleteDashboard(id, name)
-    deletePinnedItemByParam(id)
+    try {
+      await deletePinnedItemByParam(id)
+      this.props.sendNotification(pinnedItemSuccess('task', 'deleted'))
+    } catch (error) {
+      this.props.sendNotification(pinnedItemFailure(error.message, 'delete'))
+    }
   }
 
   private handleClickDashboard = event => {

src/flows/components/header/MenuButton.tsx

@@ -66,7 +66,7 @@ const MenuButton: FC<Props> = ({handleResetShare}) => {
       event('delete_notebook', {
         context: 'notebook',
       })
-      deletePinnedItemByParam(flow.id)
+      await deletePinnedItemByParam(flow.id)
       await deleteNotebook()
       setLoading(RemoteDataState.Done)
       history.push(`/orgs/${orgID}/${PROJECT_NAME_PLURAL.toLowerCase()}`)

src/tasks/components/TaskCard.tsx

@@ -157,9 +157,13 @@ export class TaskCard extends PureComponent<
     }
   }
 
-  private handleOnDelete = task => {
+  private handleOnDelete = async task => {
     this.props.onDelete(task)
-    deletePinnedItemByParam(task.id)
+    try {
+      await deletePinnedItemByParam(task.id)
+    } catch (error) {
+      this.props.sendNotification(pinnedItemFailure(error.message, 'delete'))
+    }
   }
   private get contextMenu(): JSX.Element {
     const {task, onClone, isPinned} = this.props