Page MenuHomePhabricator

[web] Add tab to payload for navigating to a chat
ClosedPublic

Authored by inka on Jan 17 2023, 5:03 AM.
Tags
None
Referenced Files
F3357747: D6276.id20991.diff
Sun, Nov 24, 1:19 AM
F3357453: D6276.diff
Sat, Nov 23, 11:45 PM
Unknown Object (File)
Sat, Nov 16, 8:13 PM
Unknown Object (File)
Mon, Nov 4, 5:59 AM
Unknown Object (File)
Mon, Oct 28, 8:11 AM
Unknown Object (File)
Oct 23 2024, 9:23 PM
Unknown Object (File)
Oct 18 2024, 1:44 AM
Unknown Object (File)
Oct 17 2024, 8:36 PM
Subscribers

Details

Summary

The function returned by useOnClickThread is used to navigate to a chat. Before, when called from a different tab that Chat, it would change the chat displayed in Chat tab, but not chagne the
tab to the Chat tab. ie if we callled it from Apps, the user would still see the Apps tab. Now calling this function will also change the tab to the Chat tab.

Test Plan

Run web app, check that in Chat tab pressing a chat item in the chat thread list still navigates correctly to the corresponding chat. Check that calling this function from different tabs changes the tab to the Chat tab, and navigates to the corresponding chat.
Check that the url gets updated to http://localhost:3000/comm/chat/thread/*

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka requested review of this revision.Jan 17 2023, 5:18 AM

Could you also include in the test plan a check if it updates the url correctly?

This revision is now accepted and ready to land.Jan 18 2023, 2:19 AM