Skip to content

Surface push registration errors instead of failing silently#844

Closed
franco-zalamena-iterable wants to merge 1 commit intoIterable:masterfrom
franco-zalamena-iterable:fix/469-surface-push-registration-errors
Closed

Surface push registration errors instead of failing silently#844
franco-zalamena-iterable wants to merge 1 commit intoIterable:masterfrom
franco-zalamena-iterable:fix/469-surface-push-registration-errors

Conversation

@franco-zalamena-iterable
Copy link
Copy Markdown

@franco-zalamena-iterable franco-zalamena-iterable commented Apr 7, 2026

Summary

  • Add error logging when device token registration fails on both iOS and Android native modules
  • Emit a handleTokenRegistrationFailedCalled event from native to JS when registration fails
  • Add onTokenRegistrationFailed callback to IterableConfig so JS code can listen for registration failures

Test plan

  • Set invalid pushIntegrationName, verify error is logged in native console
  • Set invalid pushIntegrationName, verify onTokenRegistrationFailed callback fires with reason
  • Set valid pushIntegrationName, verify registration succeeds silently (no error event)

🤖 Generated with Claude Code

When pushIntegrationName is invalid, the registerDeviceToken API call
previously failed silently. This change implements the
onTokenRegistrationFailed callback in both iOS and Android native
modules to log errors and emit events that JS code can listen to.

Fixes Iterable#469

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@franco-zalamena-iterable franco-zalamena-iterable deleted the fix/469-surface-push-registration-errors branch April 8, 2026 14:49
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.

1 participant