We don't want to display users that are friends when adding new friends, just like we don't want to do it for blocked users. To allow that, we can specify a set of excluded statuses.
Depends on D4305
Details
Details
- Reviewers
• jacek atul - Commits
- rCOMM7d5bd5b1fcab: [web] Add option to filter users by status
Check if setting excluded statuses blocks the users with these from being displayed.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Comment Actions
Check if setting excluded statuses blocks the users with these from being displayed.
As an aside, this seems like the type of thing we could relatively easily write tests for? Might be challenging to mock redux and whatnot though
Comment Actions
The easiest way of writing a test for this logic is to extract the function that computes a memo value and test it separately. If we want to test the whole component, we can also do that, but it's a matter of prioritization.