Page MenuHomePhabricator

[web] Change navInfo.selectedUserList to contains user infos
ClosedPublic

Authored by michal on May 29 2023, 6:52 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Oct 30, 3:24 PM
Unknown Object (File)
Wed, Oct 30, 3:24 PM
Unknown Object (File)
Wed, Oct 30, 3:24 PM
Unknown Object (File)
Wed, Oct 30, 3:22 PM
Unknown Object (File)
Wed, Oct 30, 12:43 AM
Unknown Object (File)
Sat, Oct 19, 3:20 PM
Unknown Object (File)
Sat, Oct 19, 12:28 AM
Unknown Object (File)
Fri, Oct 18, 11:54 PM
Subscribers

Details

Summary

The users we get from search users endpoint might not be in the userStore. Because of that this diff changes the navInfo to contain the whole userInfos instead of just ids. When selecting a new user in the chat composer we save user info in the navInfo. During redux state updates we use the previous navInfo as the source of the userInfos in the navInfoFromURL.

Depends on D8009

Test Plan

Try adding and removing users from chat composer. Check if the url and navInfo are updated correctly. Try navigating to chat/thread/new/<non-existant-user-id> and check if it is removed.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable