Skip to content

Fix callback subscription cleanup and native event lifecycle - #189

Merged
nicolas-chaix-didomi merged 2 commits into
mainfrom
fix/6890-callback-timeout
Sep 14, 2026
Merged

nicolas-chaix-didomi merged 2 commits into
mainfrom
fix/6890-callback-timeout

Conversation

@nicolas-chaix-didomi

Copy link
Copy Markdown
Contributor

Fix event listener cleanup that can cause ready/error callbacks to time out despite receiving a native event.

  • Remove only the completed or timed-out call’s subscription, preserving other pending calls.
  • On iOS, use observation state instead of bridge presence to gate events, and remove the native SDK listener during module invalidation.
  • On Android, route all events through emitDeviceEvent().

Adds regression tests.

@nicolas-chaix-didomi
nicolas-chaix-didomi merged commit c4db1b4 into main Sep 14, 2026
3 checks passed
@nicolas-chaix-didomi
nicolas-chaix-didomi deleted the fix/6890-callback-timeout branch September 14, 2026 09:28
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