Skip to content

AppsFlyer: Fix - Prevent unhandled exceptions and SDK calls when not initialized#2108

Open
cmgustavo wants to merge 1 commit intobitpay:developfrom
cmgustavo:ref/appsflyer-wrapper-01
Open

AppsFlyer: Fix - Prevent unhandled exceptions and SDK calls when not initialized#2108
cmgustavo wants to merge 1 commit intobitpay:developfrom
cmgustavo:ref/appsflyer-wrapper-01

Conversation

@cmgustavo
Copy link
Copy Markdown
Member

@cmgustavo cmgustavo commented Mar 31, 2026

RN-2412
RN-2416

Changes

  • Remove public configureResolvedDeepLinks() from the wrapper API
  • Configure wrapped deep link hosts internally during init()
  • Keep AppsFlyer state handling in the wrapper to avoid SDK calls before successful initialization

Notes

Wrapped deep link configuration is startup-time SDK setup, not a general runtime action. Keeping it inside init() simplifies call sites and avoids calling deep link configuration separately or when the SDK is not ready.

@cmgustavo cmgustavo changed the title AppsFlyer: Enhancement - Move wrapped deep link configuration into AppsFlyerWrapper AppsFlyer: Enhancement - Prevent unhandled exceptions and SDK calls when no initialized Mar 31, 2026
@cmgustavo cmgustavo changed the title AppsFlyer: Enhancement - Prevent unhandled exceptions and SDK calls when no initialized AppsFlyer: Enhancement - Prevent unhandled exceptions and SDK calls when not initialized Mar 31, 2026
@cmgustavo cmgustavo force-pushed the ref/appsflyer-wrapper-01 branch from 2d8dd1b to 19087ef Compare March 31, 2026 20:41
@cmgustavo cmgustavo changed the title AppsFlyer: Enhancement - Prevent unhandled exceptions and SDK calls when not initialized AppsFlyer: Fix - Prevent unhandled exceptions and SDK calls when not initialized Apr 1, 2026
@sentry
Copy link
Copy Markdown

sentry bot commented Apr 2, 2026

Sentry Issue: BITPAY-APP-G

@JohnathanWhite JohnathanWhite changed the base branch from master to develop April 2, 2026 22:50
Copy link
Copy Markdown
Collaborator

@gabrielbazan7 gabrielbazan7 left a comment

Choose a reason for hiding this comment

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

ACK! great job !

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