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
Unknown Object (File)
Sun, Nov 24, 1:38 AM
Unknown Object (File)
Sun, Nov 24, 1:19 AM
Unknown Object (File)
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)
Oct 28 2024, 8:11 AM
Unknown Object (File)
Oct 23 2024, 9:23 PM
Unknown Object (File)
Oct 18 2024, 1:44 AM
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
No Lint Coverage
Unit
No Test Coverage

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