Skip to content

Conversation

@BrandonStalnaker
Copy link
Collaborator

Summary

  • Updated according to new specs
    (1 is the only one that affects the core sdk)
  1. If other is a current identity, and the developer passes emailsha256 attribute with a different value from other identity to selectPlacements, log a warning to the user, and call identify with other using the value from emailsha256
  2. Never remove other from the attributes passed to selectPlacements because it's considered a normal attribute
  3. If email identity exists on the user, and emailsha256 attribute is passed to selectPlacements, do not send email attribute to the selectPlacements call. OR if email identity and other identity exist on the user, only pass emailsha256 attribute with the value from other identity

Testing Plan

  • Was this tested locally? If not, explain why.
  • Tested on simulator and with unit tests

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

@BrandonStalnaker BrandonStalnaker self-assigned this Jul 30, 2025
@BrandonStalnaker BrandonStalnaker requested a review from a team as a code owner July 30, 2025 16:32
Copy link
Contributor

@Mansi-mParticle Mansi-mParticle left a comment

Choose a reason for hiding this comment

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

LGTM

@BrandonStalnaker BrandonStalnaker merged commit 63fb357 into development Jul 30, 2025
12 checks passed
@BrandonStalnaker BrandonStalnaker deleted the feat/SQDSDKS-7555-Handle-Hashed-Email-For-Rokt-Update branch July 30, 2025 18:54
mparticle-automation added a commit that referenced this pull request Aug 4, 2025
# [8.37.0](v8.36.0...v8.37.0) (2025-08-04)

### Bug Fixes

* Make documentation generation non-blocking for releases ([1dc0c66](1dc0c66))

### Features

* Handle Hashed Email for Rokt ([#383](#383)) ([3b9e072](3b9e072))
* Update Hashed Email for Rokt ([#384](#384)) ([475e5d4](475e5d4))
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.

4 participants