Skip to content

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 (v3) web kit to use Braze’s newer initialization option for enabling user-supplied JavaScript/HTML, and hardens a Braze reserved-attribute test to avoid hard-coded years.

Changes:

  • Replace the deprecated Braze init option enableHtmlInAppMessages with allowUserSuppliedJavascript in kit initialization.
  • Update the $Age reserved attribute test to assert year-of-birth dynamically based on the current year.

Reviewed changes

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

File Description
src/BrazeKit-dev.js Switches the Braze SDK initialization option to allowUserSuppliedJavascript.
test/tests.js Makes year-of-birth expectations derived from the current year rather than a hard-coded constant.

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

@jaissica12 jaissica12 merged commit 47083a0 into master-v3 Feb 11, 2026
8 checks passed
rmi22186 pushed a commit that referenced this pull request Feb 12, 2026
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