Fix: channel.lastRead() returns null after posting a message

[ohayoyogi]

🎯 Goal

Preventing deleting last_read of current user.

🛠 Implementation details

In current implementation, getLatestMessageReadStatus function manipulates channel.state.read directly and deletes read state of current user.

  const readList = channel.state.read;
  if (currentUserId) {
    delete readList[currentUserId];
  }

This causes problems in case of using channel.lastRead() function after posting new message. Because this.state.read[userID] is set undefined after calling getLatestMessageReadStatus.

https://github.com/GetStream/stream-chat-js/blob/master/src/channel.ts#L901

To prevent this, how about copying object to avoid manipulating state directly?

🎨 UI Changes

Nothing changed.

🧪 Testing

• channel.lastRead() does not return null after posting new message.

☑️ Checklist

• [x] I have signed the Stream CLA (required)
• [x] PR targets the develop branch
• [x] Documentation is updated
• [x] New code is tested in main example apps, including all possible scenarios
  • [x] SampleApp iOS and Android
  • [x] Expo iOS and Android

[khushal87]

Hey @ohayoyogi, can you please fix the lint issues and push your code? You can run yarn run lint-fix on the root of the project to fix it.

[ohayoyogi]

I apologize to forget to lint. Could you review this again?

[khushal87]

Thanks for the PR @ohayoyogi

[stream-ci-bot]

:tada: This PR is included in version 5.30.0 :tada:

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot :package::rocket: