+
Skip to content

feat: Make it possible to leave broadcast channels #6984

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Jul 11, 2025
Merged

Conversation

Hocuri
Copy link
Collaborator

@Hocuri Hocuri commented Jul 8, 2025

Part of #6884.
The channel owner will not be notified in any way that you left, they will only see that there is one member less.

For the translated stock strings, this is what we agreed on in the group:

  • Add a new "Leave Channel" stock string (will need to be done in UIs)
  • Reword the existing "Are you sure you want to leave this group?" string to "Are you sure you want to leave?" (the options are "Cancel" and "Leave Group" / "Leave Channel", so it's clear what you are leaving) (will need to be done in the deltachat-android repo, other UIs will pick it up automatically)
  • Reword the existing "You left the group." string to "You left". (done here, I will adapt the strings in deltachat-android, too)

I adapted DC Android by pushing deltachat/deltachat-android@6df2740 to deltachat/deltachat-android#3783.

@Hocuri Hocuri requested review from link2xt and iequidoo July 8, 2025 09:03
@Hocuri Hocuri force-pushed the hoc/channels-leave branch from 988a64f to 08c3e91 Compare July 8, 2025 09:31
Hocuri added a commit to deltachat/deltachat-android that referenced this pull request Jul 8, 2025
- In a chat, if the chat is an InBroadcast, and it's not a contact
  request, then the `Leave` menu option is shown with the translated
  stock string `menu_leave_channel` as its label.
- If the user clicks on it, the confirmation dialog has
  `menu_leave_channel` (rather than `menu_leave_group`) as its positive
  option.

Counterpart of chatmail/core#6984.
Hocuri added a commit to deltachat/deltachat-android that referenced this pull request Jul 10, 2025
* Rename "broadcast list" to "channel"/"broadcast channel" both in UI and code

* feat: Add new channel types

* Update CHANGELOG.md

* adb's review

* refactor: Rename BroadcastChannel to Broadcast

* Revert accidental change

* Make it possible to leave channels

- In a chat, if the chat is an InBroadcast, and it's not a contact
  request, then the `Leave` menu option is shown with the translated
  stock string `menu_leave_channel` as its label.
- If the user clicks on it, the confirmation dialog has
  `menu_leave_channel` (rather than `menu_leave_group`) as its positive
  option.

Counterpart of chatmail/core#6984.

---------

Co-authored-by: adbenitez <asieldbenitez@gmail.com>
Co-authored-by: adb <adb@merlinux.eu>
Copy link
Collaborator

@link2xt link2xt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems it needs a multi-device test? On a second device Alice should see the info message saying that she left the group.

@Hocuri Hocuri force-pushed the hoc/channels-leave branch from 9e0205a to 89ffec3 Compare July 10, 2025 22:27
@Hocuri Hocuri force-pushed the hoc/channels-leave branch from 89ffec3 to 67c618e Compare July 11, 2025 11:41
@Hocuri Hocuri enabled auto-merge (squash) July 11, 2025 11:52
@Hocuri Hocuri merged commit 6406f30 into main Jul 11, 2025
53 of 54 checks passed
@Hocuri Hocuri deleted the hoc/channels-leave branch July 11, 2025 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载