Page MenuHomePhabricator

[native] Fix reply not working in search results
ClosedPublic

Authored by inka on May 23 2023, 6:23 AM.
Tags
None
Referenced Files
F1584540: D7942.diff
Fri, Apr 19, 3:19 AM
Unknown Object (File)
Thu, Apr 11, 12:45 PM
Unknown Object (File)
Thu, Apr 11, 12:45 PM
Unknown Object (File)
Thu, Apr 11, 12:45 PM
Unknown Object (File)
Thu, Apr 11, 12:45 PM
Unknown Object (File)
Sat, Apr 6, 11:04 AM
Unknown Object (File)
Wed, Mar 27, 11:53 PM
Unknown Object (File)
Mar 11 2024, 9:42 PM
Subscribers

Details

Summary

issue: https://linear.app/comm/issue/ENG-3881/reply-doesnt-work-in-search-screen-and-in-pinned-messages-screen
Reply from within the search screen didn't work. We need to navigate back to the thread to be able to reply.
Adding MessageSearchRouteName to threadRoutes is necessary, because we don't want chat input bar to remove its listener for input state events, which happens if activeThreadSelector returns null or an id of a different thread (see isActive variable in chat-input-bar.react.js). activeThreadSelector returns null if threadRoutes doesn't contain the current route name.
ThreadSettingsRouteName is also in threadRoutes, and the search screen is used in a similar fashion to thread settings screen.

Test Plan

Pressed "reply" on the search screen. Checked that the correct thread was opened and the input bar message was prepended with the reply.

Diff Detail

Repository
rCOMM Comm
Branch
inka/search_backend
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

inka requested review of this revision.May 23 2023, 6:41 AM
inka planned changes to this revision.May 23 2023, 8:15 AM
native/chat/text-message-tooltip-modal.react.js
38 ↗(On Diff #26903)

We already have route variable, we can probably use it.

Thanks for explaining everything!

This revision is now accepted and ready to land.May 23 2023, 11:57 AM