Removing channel members doesn't send a system message

Open
#163 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
58/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Stale
Tech stack
php
Domain
api, backend

Research direction

Start in Channel.php, comparing removeMembers with addMembers and the documented channel-member removal options. Verify how the optional options are passed through and confirm that removal can produce the documented system message behavior; update or run the relevant existing checks if present.

Written by the indexing model from the issue text.

Description

When adding users to a channel, you can pass on the option to also send a system message to the channel (I use it to notifiy channel members with a message like " has been added to this channel". The documentation cover this pretty well. The documentation also states that this is possible when removing members from a channel.

Image

However, the removeMembers function in Channel.php only accepts an array of userIds, and nothing else. Comparing it to addMembers, it seems like it's missing a second parameter $options to be able to pass on options such as a system message.

Image

Would it be possible to add the $options optional parameter in the removeMembers function, so that it behaves similarly to the addMembers function?

Dominant language
PHP
Stars
29
Forks
9
Avg merge
11h 38m
Merged PRs (30d)
2

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from GetStream/stream-chat-php

All issues in GetStream/stream-chat-php

Similar issues

More PHP issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.