Skip to content

Conversation

@isekovanic
Copy link
Contributor

🎯 Goal

This PR fixes an unfortunate issue with state not being properly synchronized in certain situations (typically on lower end devices), where the updates of our processedMessageList would be ahead of the updates of the previous and next message store.

Even thought there's a dependency on processedMessageList, since renderItem is already memoized and the message references are stable from the work done on the MessageList the past few months, this does not in fact degrade performance while we focus on correctness.

🛠 Implementation details

🎨 UI Changes

iOS
Before After
Android
Before After

🧪 Testing

☑️ Checklist

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

@Stream-SDK-Bot
Copy link
Contributor

Stream-SDK-Bot commented Jan 20, 2026

SDK Size

title develop branch diff status
js_bundle_size 339 KB 339 KB 0 B 🟢

@isekovanic isekovanic merged commit 6996b50 into develop Jan 20, 2026
5 checks passed
@isekovanic isekovanic deleted the fix/state-synchronisity-mismatch branch January 20, 2026 21:04
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.

5 participants