Skip to content

common: Add chat message removal flows and APIs#1008

Open
richtrwhereby wants to merge 1 commit into
mainfrom
richardtibbett/pan-2634-sdk-remove-a-message
Open

common: Add chat message removal flows and APIs#1008
richtrwhereby wants to merge 1 commit into
mainfrom
richardtibbett/pan-2634-sdk-remove-a-message

Conversation

@richtrwhereby
Copy link
Copy Markdown
Contributor

@richtrwhereby richtrwhereby commented May 25, 2026

Description

Add removeChatMessage API to both Core and Browser SDKs. Handle chat message removal responses from the server (i.e. mark chat messages with removed: true).

Testing

  1. Run pnpm build && pnpm dev
  2. When storybook starts, navigate to "Stories" > "Custom UI" > "Room connection only"
  3. Join a Whereby room
  4. Send a chat message in the room via the storybook UI
  5. When the chat message appears, click "Remove" next to that message.
  6. Verify that the message is removed from the room and the message is displayed with strikethrough formatting.
  7. When storybook starts, navigate to "Stories" > "Custom UI" > "Room connection With Host Controls"
  8. Join a Whereby room
  9. Send a chat message in the room via the storybook UI
  10. When the chat message appears, click "Remove" next to that message.
  11. Join the same room with another user and send a chat message from that user.
  12. When the chat message appears in the storybook UI, click "Remove" next to that message.
  13. Verify that a host is able to remove any/all messages from the room and all removed messages are displayed with strikethrough formatting.

Checklist

  • My code follows the project's coding standards.
  • Prefixed the PR title and commit messages with the service or package name
  • I have written unit tests (if applicable).
  • I have updated the documentation (if applicable).
  • By submitting this pull request, I confirm that my contribution is made
    under the terms of the MIT license.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 25, 2026

🦋 Changeset detected

Latest commit: c6ca91d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
@whereby.com/browser-sdk Patch
@whereby.com/media Patch
@whereby.com/core Minor
@whereby.com/assistant-sdk Patch
@whereby.com/react-native-sdk Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@richtrwhereby richtrwhereby force-pushed the richardtibbett/pan-2634-sdk-remove-a-message branch from 447d9d6 to c6ca91d Compare May 25, 2026 15:01
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.

1 participant