Introduce component grouping users and rendering list based on search results in AddMembersModal.
I used common logic with native app, where only users from parent thread have "notice" set to "undefined". It's the reason, why after grouping the title: "Users in parent thread" is added manually.
Details
Details
The list can be tested after introducing AddMembersModal
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
web/modals/threads/members/add-members-list.react.js | ||
---|---|---|
20–21 ↗ | (On Diff #11511) | We don't memoize these so all the following memoizations will recompute after each render. |
37–45 ↗ | (On Diff #11511) | I think this code might become more readable when:
_toPairs(groupedAvailableUsersList) .filter(group => group[0] !== 'undefined') .sort((a, b) => a[0].localeCompare(b[0]))
|
web/modals/threads/members/add-members-list.react.js | ||
---|---|---|
13 | Can probably use $ReadOnlySet<string> here instead? |