Skip to content
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

chore: fix branch initialization issue #330

Merged
merged 3 commits into from
Mar 11, 2024

Conversation

saeedbashir
Copy link
Contributor

@saeedbashir saeedbashir commented Mar 8, 2024

I've followed the Branch SDK Integration SDK steps by step. But due to some unknown issues, the latest Branch SDK 3.2.0 isn't working as expected. Even it's not giving initialization call back.

Me and @rnr spent a reasonable amount of time on it and are still not able to figure out why it's not working. The same codebase is working perfectly fine for SDK version 2.2.0.

For the MVP, we don't have the luxury of time to reach out to the branch support team and diagnose the issue. For the time being, I've downgraded the branch SDK to 2.2.0 so we can launch the MVP. We'll look into it post MVP.

Note: We stopped getting an initialization callback for the edX identifier org.edx.mobile. It works fine for the openedX identifier. The down graded version is working for both edX and openedX identifiers.

@saeedbashir saeedbashir changed the title chore: configure branch via info.plist chore: fix branch initialization issue Mar 11, 2024
Copy link
Contributor

@volodymyr-chekyrta volodymyr-chekyrta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @saeedbashir @rnr 👍

@saeedbashir saeedbashir merged commit 5821f61 into openedx:develop Mar 11, 2024
3 checks passed
@saeedbashir saeedbashir deleted the saeed/branch_fix branch March 11, 2024 14:41
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