Page MenuHomePhabricator
Differential D13542

[lib] Don't display notFriendNotice for the viewer
ClosedPublic
Actions

Authored by ashoat on Sep 30 2024, 9:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 10, 4:20 PM
Unknown Object (File)
Fri, Nov 8, 3:01 AM
Unknown Object (File)
Fri, Nov 8, 3:01 AM
Unknown Object (File)
Fri, Nov 1, 4:46 PM
Unknown Object (File)
Fri, Nov 1, 5:37 AM
Unknown Object (File)
Sun, Oct 20, 3:58 PM
Unknown Object (File)
Oct 19 2024, 12:14 PM
Unknown Object (File)
Oct 19 2024, 10:00 AM
View All 32 Files
Subscribers
None

Details

Reviewers
tomek
kamil
inka
Commits
rCOMM31793e3ec987: [lib] Don't display notFriendNotice for the viewer
Summary

Prior to the changes in D13540, usePotentialMemberItems never got the viewer as an input, so it didn't need to consider it.

It currently sets notFriendNotice for the viewer, which doesn't make sense. While it's true that the viewer is not a friend with themselves, we have no reason to display that notice to the user.

Depends on D13541

Test Plan

Confirm that notFriendNotice no longer appears for the viewer (the viewer's entry in the list of users that can be added to the chat)

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
lib/
shared/
12 lines

Diff 44723

View Options

lib/shared/search-utils.js

Loading...