Skip to content

Avoid import-time crash from new NativeEventEmitter(Wallet) - #67

Merged
JakubKorytko merged 2 commits into
Expensify:mainfrom
MegaManSec:pr/jr-14-12361e0
Sep 30, 2026
Merged

JakubKorytko merged 2 commits into
Expensify:mainfrom
MegaManSec:pr/jr-14-12361e0

Conversation

@MegaManSec

Copy link
Copy Markdown
Contributor

No description provided.

@JakubKorytko JakubKorytko left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

fixes the iOS crash when the module isn't linked, but introduces a regression on Android. LGTM when addressed

Comment thread src/index.tsx Outdated

@JakubKorytko JakubKorytko left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Re-tested, LGTM

@JakubKorytko

Copy link
Copy Markdown
Member

@MegaManSec, could you sign and rebase the last commit? I will re-approve but it lost its signature

@JakubKorytko

Copy link
Copy Markdown
Member

thank you!

@JakubKorytko
JakubKorytko merged commit 24c6a4a into Expensify:main Sep 30, 2026
7 checks passed
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