[native] Add log for when all_thread_members is used for a thread missing from ThreadStore
ClosedPublic
Actions

Authored by ashoat on Sep 23 2024, 10:18 PM.

Details

Reviewers

Commits

rCOMMda25916ca609: [native] Add log for when all_thread_members is used for a thread missing from…

Summary

I spent some time figuring out this issue, which was previously addressed by @kamil when working on D13328. Adding this log to make it easier for the next person.

Depends on D13431

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Sep 23 2024, 10:18 PM

Herald added a subscriber: tomek. · View Herald TranscriptSep 23 2024, 10:18 PM

ashoat added a child revision: D13433: [lib] Include targetUserID in DMUpdateRelationshipOperation.Sep 23 2024, 10:20 PM

Harbormaster completed remote builds in B31789: Diff 44455.Sep 23 2024, 10:34 PM

ashoat requested review of this revision.Sep 23 2024, 10:34 PM

kamil accepted this revision.Sep 24 2024, 12:45 AM

This revision is now accepted and ready to land.Sep 24 2024, 12:45 AM

Closed by commit rCOMMda25916ca609: [native] Add log for when all_thread_members is used for a thread missing from… (authored by ashoat). · Explain WhySep 24 2024, 3:04 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMMda25916ca609: [native] Add log for when all_thread_members is used for a thread missing from….

Revision Contents
Changeset List

Path

Size

lib/

shared/

dm-ops/

dm-op-utils.js

8 lines

Diff 44455

View Options

lib/shared/dm-ops/dm-op-utils.js

[native] Add log for when all_thread_members is used for a thread missing from ThreadStoreClosedPublicActions