HomePhabricator
Diffusion Comm 8edec5cb6dad

[web] introduce useSortedENSResolvedUsers

Description

[web] introduce useSortedENSResolvedUsers

Summary: Got some feedback in D10822 about sorting the users AFTER resolving ENS names. Noticed we needed to make this change in two places in AddUsersList + we will probably want to reuse this logic for the chat members list, so created this hook to factor out the shared logic

Test Plan:
Please see the screenshots below

Before:

Screenshot 2024-01-31 at 2.30.29 PM.png (2×3 px, 798 KB)

Screenshot 2024-01-31 at 2.30.41 PM.png (2×3 px, 781 KB)

After:

Screenshot 2024-01-31 at 2.30.58 PM.png (2×3 px, 781 KB)

Screenshot 2024-01-31 at 2.30.48 PM.png (2×3 px, 777 KB)

Reviewers: atul, inka

Reviewed By: atul

Subscribers: ashoat, tomek

Differential Revision: https://phab.comm.dev/D10897

Details

Provenance
ginsuAuthored on Jan 31 2024, 12:35 PM
Reviewer
atul
Differential Revision
D10897: [web] introduce useSortedENSResolvedUsers
Parents
rCOMM85f4aa485848: [lib] Split `WebInitialReduxStateResponse` into `Client/Server` variants
Branches
Unknown
Tags
Unknown