[lib] Use the useBroadcastOlmMessage hook in account deletion
ClosedPublic
Actions

Authored by bartek on Aug 6 2024, 8:04 AM.

Details

Reviewers

kamil

Commits

rCOMM89096e302465: [lib] Use the useBroadcastOlmMessage hook in account deletion

Summary

Updated useBroadcastAccountDeletion to use the new useBroadcastOlmMessage hook.
Also renamed broadcastToOwnDevices to includeOwnDevices because the old name was confusing.

Depends on D12994, D12995

Test Plan

Repeated test plan from D12944

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Aug 6 2024, 8:04 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptAug 6 2024, 8:04 AM

bartek added a child revision: D12997: [lib] Use the useBroadcastOlmMessage hook in logout actions.Aug 6 2024, 8:06 AM

Harbormaster completed remote builds in B30951: Diff 43166.Aug 6 2024, 8:26 AM

bartek published this revision for review.Aug 6 2024, 8:36 AM

bartek added inline comments.

lib/hooks/peer-list-hooks.js
216–218	Previous name was confusing. Default value was also counterintuitive. Opted out of it. Also, addressed https://phab.comm.dev/D12944?id=42964#inline-75242