From 41ea9ee15b6692d23f9a8fefacbec4715039cf10 Mon Sep 17 00:00:00 2001 From: Scott Nonnenberg Date: Wed, 20 Feb 2019 11:57:14 -0800 Subject: [PATCH] Ensure that we show error'd recipients in message details screen --- js/models/messages.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/js/models/messages.js b/js/models/messages.js index 7ba1794e0a..5fddb62e03 100644 --- a/js/models/messages.js +++ b/js/models/messages.js @@ -629,11 +629,15 @@ return accumulator; }, Object.create(null)); + // We include numbers we didn't successfully send to so we can display errors. // Older messages don't have the recipients included on the message, so we fall // back to the conversation's current recipients const phoneNumbers = this.isIncoming() ? [this.get('source')] - : this.get('sent_to') || this.getConversation().getRecipients(); + : _.union( + this.get('sent_to') || [], + this.get('recipients') || this.getConversation().getRecipients() + ); // This will make the error message for outgoing key errors a bit nicer const allErrors = (this.get('errors') || []).map(error => {