Skip to content

Fix: Replace enableHtmlInAppMessages deprecated method with allowUserSuppliedJavascript (v4)#61

Merged
jaissica12 merged 2 commits intomaster-v4from
fix/SDKE-882-fix-deprecated-method-v4
Feb 11, 2026
Merged

Fix: Replace enableHtmlInAppMessages deprecated method with allowUserSuppliedJavascript (v4)#61
jaissica12 merged 2 commits intomaster-v4from
fix/SDKE-882-fix-deprecated-method-v4

Conversation

@jaissica12
Copy link
Contributor

Instructions

  1. PR target branch should be against development
  2. PR title name should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-title-check.yml
  3. PR branch prefix should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-branch-check-name.yml

Summary

  • Replacing deprecated enableHtmlInAppMessages setting with new one allowUserSuppliedJavascript

Testing Plan

  • Was this tested locally? If not, explain why.
  • {explain how this has been tested, and what, if any, additional testing should be done}

Reference Issue (For mParticle employees only. Ignore if you are an outside contributor)

Copy link
Collaborator

@alexs-mparticle alexs-mparticle left a comment

Choose a reason for hiding this comment

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

I love the attempt to make this dynamic, but I would keep the comment explaining the logic for the 10 year difference, since it looks like this is just using 10 as an arbitrary number.

@jaissica12 jaissica12 merged commit ca2d021 into master-v4 Feb 11, 2026
2 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