Skip to content

Add setBranchKey support and refactor Branch key handling #450

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

VasanthanRajTouchtier
Copy link

Introduced setBranchKey method to FlutterBranchSdk for dynamic Branch key assignment, and updated Android and iOS plugins to support this method. Deprecated static Branch key configuration in AndroidManifest.xml and Info.plist in favor of runtime assignment. Refactored and formatted code for improved readability and maintainability, and updated example usage accordingly.

Introduced setBranchKey method to FlutterBranchSdk for dynamic Branch key assignment, and updated Android and iOS plugins to support this method. Deprecated static Branch key configuration in AndroidManifest.xml and Info.plist in favor of runtime assignment. Refactored and formatted code for improved readability and maintainability, and updated example usage accordingly.
@RodrigoSMarques
Copy link
Owner

RodrigoSMarques commented Aug 9, 2025

Thank you for your contribution.

Adding the setBranchKey method involves delaying SDK initialization in native code.

From version 5.x to version 6.x, we implemented an initialization delay, which caused a large number of issues, especially on iOS.

The issue was fixed from version 6.x to version 7.x, removing the initialization delay.

In the past, I've tried implementing key configuration and TestKey usage via a plugin.

It works in some scenarios, but not in others.

From experience, I'm not confident about implementing any changes to the SDK initialization flow.

@RodrigoSMarques RodrigoSMarques changed the base branch from master to dev August 10, 2025 17:20
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.

2 participants