Page MenuHomePhabricator

[lib] fix `useThreadSearchIndex` to avoid ignoring private thread
ClosedPublic

Authored by kamil on Wed, Sep 25, 6:11 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Sep 25, 12:22 PM
Unknown Object (File)
Wed, Sep 25, 12:22 PM
Unknown Object (File)
Wed, Sep 25, 12:21 PM
Unknown Object (File)
Wed, Sep 25, 12:04 PM
Subscribers
None

Details

Summary

ENG-9378.

This code was ignoring private threads and as a result, causing that private thread not to show up.
On prod, it works only for old users where by default thread name was the user's name, after changing the name in settings it is not working anymore. Right now, when creating an account name is not set for the private thread, it's an optional field and not set to own username - that is why this is not working for dev where we have new users.

Test Plan

Searching for my username works (even if I changed name of my private thread).

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Wed, Sep 25, 6:12 AM
kamil edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Wed, Sep 25, 6:30 AM

You're right – my PRIVATE thread is named "ashoat", which explains why it was working for me