Page MenuHomePhabricator

[web] Add option to filter users by status
ClosedPublic

Authored by tomek on Jun 21 2022, 5:19 AM.
Tags
None
Referenced Files
F3488041: D4306.diff
Wed, Dec 18, 8:49 AM
Unknown Object (File)
Fri, Dec 6, 3:33 PM
Unknown Object (File)
Nov 17 2024, 6:28 AM
Unknown Object (File)
Nov 9 2024, 7:59 AM
Unknown Object (File)
Nov 9 2024, 2:27 AM
Unknown Object (File)
Oct 14 2024, 12:46 AM
Unknown Object (File)
Oct 14 2024, 12:46 AM
Unknown Object (File)
Oct 14 2024, 12:46 AM

Details

Summary

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

Test Plan

Check if setting excluded statuses blocks the users with these from being displayed.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tomek requested review of this revision.Jun 21 2022, 5:24 AM

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

This revision is now accepted and ready to land.Jun 21 2022, 10:17 AM
In D4306#121576, @atul wrote:

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

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.

This revision was automatically updated to reflect the committed changes.