Skip to content

Make InAppMessage.customPayload optional in type definition#570

Closed
franco-zalamena-iterable wants to merge 1 commit intoIterable:mainfrom
franco-zalamena-iterable:fix/538-custom-payload-optional-type
Closed

Make InAppMessage.customPayload optional in type definition#570
franco-zalamena-iterable wants to merge 1 commit intoIterable:mainfrom
franco-zalamena-iterable:fix/538-custom-payload-optional-type

Conversation

@franco-zalamena-iterable
Copy link
Copy Markdown

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

Summary

  • Make customPayload property optional in InAppMessage interface
  • Accurately reflects runtime behavior where customPayload can be undefined

Test plan

  • TypeScript compilation passes
  • Existing tests pass

🤖 Generated with Claude Code

The customPayload property can be undefined when no RAW JSON is
configured in the Iterable dashboard. Update the type to accurately
reflect runtime behavior.

Fixes Iterable#538

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.45%. Comparing base (190a168) to head (b9b25f0).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #570   +/-   ##
=======================================
  Coverage   74.45%   74.45%           
=======================================
  Files          57       57           
  Lines        2736     2736           
  Branches      798      798           
=======================================
  Hits         2037     2037           
  Misses        698      698           
  Partials        1        1           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@franco-zalamena-iterable franco-zalamena-iterable deleted the fix/538-custom-payload-optional-type branch April 8, 2026 14:47
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