This is a fix for ENG-9453.
Alternative is introducing another queue in useSendComposableDMOperation to make sure we're always calculating replies count on the newest thread info, but I prefer this solution and it's cleaner.
Differential D13661
[lib] move calling `getThreadUpdatesForNewMessages` to `reduceThreadInfos` kamil on Oct 9 2024, 7:07 AM. Authored by Tags None Referenced Files
Subscribers
Details This is a fix for ENG-9453. Alternative is introducing another queue in useSendComposableDMOperation to make sure we're always calculating replies count on the newest thread info, but I prefer this solution and it's cleaner. Send a lot of messages and checking replies count - did test multiple times as there is no single way to reproduce it.
Diff Detail
Event Timeline
Comment Actions I'm going to land it now to fix it - but if the comment is still not okay, I can update this later as a follow-up. |