Skip to content

Fix: Replace enableHtmlInAppMessages deprecated method with allowUserSuppliedJavascript#59

Merged
jaissica12 merged 3 commits intomasterfrom
fix/SDKE-882-replace-deprecated-method
Feb 11, 2026
Merged

Fix: Replace enableHtmlInAppMessages deprecated method with allowUserSuppliedJavascript#59
jaissica12 merged 3 commits intomasterfrom
fix/SDKE-882-replace-deprecated-method

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

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Braze web kit to align with newer Braze SDK initialization options by swapping a deprecated init flag, and stabilizes a previously year-dependent unit test.

Changes:

  • Replace options.enableHtmlInAppMessages with options.allowUserSuppliedJavascript in the Braze init options.
  • Update the $Age-to-year-of-birth unit test to compute the expected year dynamically.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
test/tests.js Makes the $Age reserved attribute test compute expected year-of-birth dynamically to avoid annual breakage.
src/BrazeKit-dev.js Switches the Braze init options field to allowUserSuppliedJavascript.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jaissica12 jaissica12 merged commit dd5263e into master 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