Skip to content

fix: show correct read status on the message status component for a message#3023

Merged
khushal87 merged 2 commits intodevelopfrom
fix-message-status-read-status
Mar 14, 2025
Merged

fix: show correct read status on the message status component for a message#3023
khushal87 merged 2 commits intodevelopfrom
fix-message-status-read-status

Conversation

@khushal87
Copy link
Copy Markdown
Contributor

@khushal87 khushal87 commented Mar 14, 2025

The PR's goal is to show the correct read status(readBy number) on the message status component.

For 1:1 channels/group channels with only two users:

  • We will no longer show the read count as it doesn't make sense since the only user who can read the message is the other user.
  • We only update the check mark to double if the other user has read the message in the channel. This was buggy before. ⚠️

For channels with more group members:

  • We will show a single check mark until no one has read the message in the channel. We do not show any read count until then.
  • Update it to double as soon as someone reads the message and then show the count of how many other users have read the message. Previously, it included my read count as well, which was buggy. ⚠️

This is inline with the iOS SDK behaviour.

Linear issue - https://linear.app/stream/issue/RN-162/show-correct-read-status-on-the-message-status-component

@Stream-SDK-Bot
Copy link
Copy Markdown
Contributor

Stream-SDK-Bot commented Mar 14, 2025

SDK Size

title develop branch diff status
js_bundle_size 467 KB 467 KB 0 B 🟒

@khushal87 khushal87 merged commit 6a1e9f6 into develop Mar 14, 2025
6 checks passed
@khushal87 khushal87 deleted the fix-message-status-read-status branch March 14, 2025 13:08
@github-actions github-actions bot mentioned this pull request Mar 17, 2025
@stream-ci-bot
Copy link
Copy Markdown
Contributor

πŸŽ‰ This PR is included in version 6.7.0 πŸŽ‰

The release is available on:

Your semantic-release bot πŸ“¦πŸš€

@khushal87 khushal87 linked an issue Mar 18, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

How to prevent auto read my own message [πŸ›] Bug Report Title - Read receipts is off by 1, counts user that sent message as having viewed

4 participants