Paths

Table of Contentst

Differential D13487

[lib] fix thick thread showing multiple times in thread list
ClosedPublic
Actions

Authored by kamil on Sep 26 2024, 10:49 AM.

Tags

None

Referenced Files

None

Subscribers

None

Details

Reviewers

ashoat
tomek

Commits

rCOMMf1a29a5027cc: [lib] fix thick thread showing multiple times in thread list

Summary

Test Plan

Search for users which has a personal thick thread and should show up only once

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Sep 26 2024, 10:49 AM

kamil held this revision as a draft.

kamil added a child revision: D13488: [lib] rename `userSupportsThickThreads` -> `userHasDeviceList`.Sep 26 2024, 10:54 AM

kamil added a parent revision: D13486: [lib] implement fetching missing peers when generating P2P messages.Sep 26 2024, 11:00 AM

Harbormaster completed remote builds in B31877: Diff 44595.Sep 26 2024, 11:06 AM

kamil published this revision for review.Sep 26 2024, 11:22 AM

ashoat accepted this revision.Sep 26 2024, 2:54 PM

This revision is now accepted and ready to land.Sep 26 2024, 2:54 PM

ashoat added inline comments.Sep 27 2024, 4:00 PM

lib/selectors/user-selectors.js
186–187 ↗	(On Diff #44595)	Instead of this, can we actually use `threadTypeIsPersonal`? I'd like to be able to `git grep GENESIS_PERSONAL` and only get results that are specific to `GENESIS_PERSONAL`

kamil added inline comments.Sep 30 2024, 11:44 AM

lib/selectors/user-selectors.js
186–187 ↗	(On Diff #44595)	good call

use threadTypeIsPersonal

This revision was landed with ongoing or failed builds.Sep 30 2024, 11:58 AM

Closed by commit rCOMMf1a29a5027cc: [lib] fix thick thread showing multiple times in thread list. · Explain Why

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMMf1a29a5027cc: [lib] fix thick thread showing multiple times in thread list.

Harbormaster completed remote builds in B31927: Diff 44672.Sep 30 2024, 12:06 PM

Revision Contents
Changeset List

Path

Size

lib/

selectors/

user-selectors.js

4 lines

Diff 44675

lib/selectors/user-selectors.js

Loading...