Skip to content

common: Add chat message reply flows and APIs#1010

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

common: Add chat message reply flows and APIs#1010
richtrwhereby wants to merge 1 commit into
richardtibbett/pan-2634-sdk-remove-a-messagefrom
richardtibbett/pan-2633-sdk-reply-to-message

Conversation

@richtrwhereby
Copy link
Copy Markdown
Contributor

@richtrwhereby richtrwhereby commented May 26, 2026

Description

Add an optional parentId value to the sendChatMessage API in both Core and Browser SDKs. By passing the id of the message that we want to reply to in this parentId argument, we can construct threaded messaging within rooms.

Related Issue:

Built on top of #1008 due to shared code space.

Testing

  1. Run pnpm build && pnpm dev
  2. When storybook starts, navigate to "Stories" > "Custom UI" > "Room connection only"
  3. Join a Whereby room from this UI
  4. Additionally, join the same Whereby room from the standard Whereby interface (non-SDK).
  5. Send a chat message in the room via the storybook UI
  6. When the chat message appears, select "Reply to: [text]" from the "Send to room" select dropdown and click "Send message".
  7. Verify that the message appears as a 'reply' to the first message in the standard Whereby interface.

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 26, 2026

🦋 Changeset detected

Latest commit: b86c66d

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/media Minor
@whereby.com/browser-sdk Minor
@whereby.com/core Patch
@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

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