Chat messages are not correctly grouped if system flag changes
Categories
(Thunderbird :: Instant Messaging, defect)
Tracking
(thunderbird100 fixed, thunderbird101 affected)
People
(Reporter: freaktechnik, Assigned: freaktechnik)
Details
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
wsmwk
:
approval-comm-beta+
|
Details | Review |
When appending chat messages that have identical flags except for the system flag changing, we incorrectly group them. Instead, a change of the system flag should always start a new group. We have not seen this often, since usually system messages have a who
that isn't attributed to a participant that can send normal messages.
Assignee | ||
Comment 1•2 years ago
|
||
Assignee | ||
Updated•2 years ago
|
Pushed by nicolai@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/cfd847f2f3b8
Fix chat message grouping and replacement with system messages. r=clokep
Assignee | ||
Comment 3•2 years ago
|
||
Comment on attachment 9271938 [details]
Bug 1764329 - Fix chat message grouping and replacement with system messages. r=clokep
[Approval Request Comment]
Regression caused by (bug #): possibly always been an issue, just never surfaced as much
User impact if declined: Chat messages might be displayed without a sender, or not collapse properly etc.
Testing completed (on c-c, etc.): tested on c-c, has a mochitest (applies to c-b)
Risk to taking this patch (and alternatives if risky): Messes a lot with how the conversation browser displays chat messages, but has automated tests as well as adding various measures to make the code more tolerant in general. The alternative is to wait a week for a whole new beta to be branched.
Comment 4•2 years ago
|
||
Comment on attachment 9271938 [details]
Bug 1764329 - Fix chat message grouping and replacement with system messages. r=clokep
[Triage Comment]
Approved for beta despite the risk ... because beta 101.0b1 is going to be a bit of big splash we hope, so better to find out now, than next week, if there are problems
Comment 5•2 years ago
|
||
bugherder uplift |
Thunderbird 100.0b4:
https://hg.mozilla.org/releases/comm-beta/rev/7dcef20b6ed0
Description
•