From 3f19bc51bb0d9a4c1232435d687c244c2ad92f35 Mon Sep 17 00:00:00 2001 From: Eric Bailey Date: Wed, 15 May 2024 11:01:28 -0500 Subject: [PATCH] Remove todos --- src/screens/Messages/Conversation/MessageListError.tsx | 1 - src/state/messages/convo/agent.ts | 7 +------ src/state/messages/convo/types.ts | 4 ---- 3 files changed, 1 insertion(+), 11 deletions(-) diff --git a/src/screens/Messages/Conversation/MessageListError.tsx b/src/screens/Messages/Conversation/MessageListError.tsx index 38a63b0f12..c6e246a3fb 100644 --- a/src/screens/Messages/Conversation/MessageListError.tsx +++ b/src/screens/Messages/Conversation/MessageListError.tsx @@ -26,7 +26,6 @@ export function MessageListError({ msg`This chat was disconnected due to a network error.`, ), [ConvoItemError.HistoryFailed]: _(msg`Failed to load past messages.`), - [ConvoItemError.PendingFailed]: _(msg`Failed to send message(s).`), }[item.code] }, [_, item.code]) diff --git a/src/state/messages/convo/agent.ts b/src/state/messages/convo/agent.ts index 4338b698bd..420eff3497 100644 --- a/src/state/messages/convo/agent.ts +++ b/src/state/messages/convo/agent.ts @@ -749,14 +749,9 @@ export class Convo { id: tempId, message, }) - // remove on each send, it might go through now without user having to click - // TODO do need htis? - this.footerItems.delete(ConvoItemError.PendingFailed) this.commit() - // TODO maybe ignore if failed? - - if (!this.isProcessingPendingMessages) { + if (!this.isProcessingPendingMessages && !this.pendingFailed) { this.processPendingMessages() } } diff --git a/src/state/messages/convo/types.ts b/src/state/messages/convo/types.ts index 2fcfacabc8..3fb0eb6ad3 100644 --- a/src/state/messages/convo/types.ts +++ b/src/state/messages/convo/types.ts @@ -35,10 +35,6 @@ export enum ConvoItemError { * Error fetching past messages */ HistoryFailed = 'historyFailed', - /** - * Error sending new message - */ - PendingFailed = 'pendingFailed', } export enum ConvoErrorCode {