[web] add missing warning about not leaving the thread when being the only admin
ClosedPublic
Actions

Authored by kamil on Sep 4 2025, 10:04 AM.

Details

Reviewers

tomek
bartek

Commits

rCOMM4806768cff1f: [web] add missing warning about not leaving the thread when being the only admin

Summary

ENG-11179.

We already have it on native in here. Adding the same functionality on web to make it symmetric. We're going to use it for Farcaster threads too.

Web:

Existing code on native:

Test Plan

Test leaving thread when being/not being only admin.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Sep 4 2025, 10:04 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptSep 4 2025, 10:04 AM

kamil published this revision for review.Sep 4 2025, 10:06 AM

kamil edited the summary of this revision. (Show Details)

bartek accepted this revision.Sep 4 2025, 10:09 AM

This revision is now accepted and ready to land.Sep 4 2025, 10:09 AM

Harbormaster completed remote builds in B35627: Diff 50157.Sep 4 2025, 10:22 AM

kamil added a child revision: D15290: [lib][web][native] extract changing thread member role to protocol.Sep 4 2025, 10:59 AM

Closed by commit rCOMM4806768cff1f: [web] add missing warning about not leaving the thread when being the only admin. · Explain WhySep 5 2025, 9:12 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM4806768cff1f: [web] add missing warning about not leaving the thread when being the only admin.

Revision Contents
Changeset List

Path

Size

web/

chat/

thread-menu.react.js

17 lines

modals/

threads/

confirm-leave-thread-modal.react.js

19 lines

Diff 50172

View Options

web/chat/thread-menu.react.js