Skip to content

fix: expose default native handlers#3704

Merged
isekovanic merged 1 commit into
developfrom
fix/expose-default-native-handlers
Jul 3, 2026
Merged

fix: expose default native handlers#3704
isekovanic merged 1 commit into
developfrom
fix/expose-default-native-handlers

Conversation

@isekovanic

Copy link
Copy Markdown
Contributor

🎯 Goal

This PR addresses this GH issue.

While exposing a prop might be slightly more convenient, our native handlers have a ton of arguments we don't want to expose every single one of them separately as it will cause an argument storm.

Instead, we expose the default native handlers and allow integrators to use them as they see fit.

For example, they could invoke them with different properties, make them conditionally depend on configuration and similar.

It would also make it easier to conditionally override the handlers in certain scenarios.

Runtime passing of different arguments should still be done on the callsites themselves, this approach is more of a global switch to something else if necessary.

🛠 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

@isekovanic isekovanic requested review from oliverlaz and szuperaz July 3, 2026 15:55
@Stream-SDK-Bot

Copy link
Copy Markdown
Contributor

SDK Size

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

@isekovanic isekovanic merged commit 9e89a66 into develop Jul 3, 2026
10 checks passed
@isekovanic isekovanic deleted the fix/expose-default-native-handlers branch July 3, 2026 16:03
@github-actions github-actions Bot mentioned this pull request Jul 3, 2026
6 tasks
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.

3 participants