colin/Signal-Desktop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Ensure that we show error'd recipients in message details screen

Browse Source
This commit is contained in:
Scott Nonnenberg
2019-02-20 11:57:14 -08:00
parent ad53423e0a
commit 41ea9ee15b
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
js/models/messages.js
View File

@@ -629,11 +629,15 @@
return accumulator; return accumulator;
}, Object.create(null)); }, 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 // Older messages don't have the recipients included on the message, so we fall
// back to the conversation's current recipients // back to the conversation's current recipients
const phoneNumbers = this.isIncoming() const phoneNumbers = this.isIncoming()
? [this.get('source')] ? [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 // This will make the error message for outgoing key errors a bit nicer
const allErrors = (this.get('errors') || []).map(error => { const allErrors = (this.get('errors') || []).map(error => {