Page MenuHomePhabricator

[web] Trigger sendCallbacks on multimedia messages
ClosedPublic

Authored by ashoat on Apr 12 2023, 6:21 PM.
Tags
None
Referenced Files
F3505581: D7407.diff
Fri, Dec 20, 1:57 PM
Unknown Object (File)
Thu, Dec 5, 2:40 AM
Unknown Object (File)
Thu, Dec 5, 2:40 AM
Unknown Object (File)
Nov 5 2024, 4:17 AM
Unknown Object (File)
Nov 5 2024, 4:17 AM
Unknown Object (File)
Nov 1 2024, 8:33 AM
Unknown Object (File)
Nov 1 2024, 8:33 AM
Unknown Object (File)
Nov 1 2024, 8:33 AM
Subscribers

Details

Summary

This is part 2 of my comment on D7388:

Part of the issue is that sendCallbacks were only getting triggered for text messages on web. I believe this was an oversight... it is triggered for media on native, and it needs to be triggered for media on web as well in order for the "search" to be closed when a thread is created with a media message. This will be addressed in the next diff.

Depends on D7388

Test Plan

Create a thread by sending a media message on web and confirm that the "serach" experienced closes (it didn't before this diff)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable