Skip to content

feat: Add Missing User Identities - #20

Merged
BrandonStalnaker merged 1 commit into
masterfrom
feat/Add-Missing-User-Identities
Aug 6, 2026
Merged

feat: Add Missing User Identities#20
BrandonStalnaker merged 1 commit into
masterfrom
feat/Add-Missing-User-Identities

Conversation

@BrandonStalnaker

Copy link
Copy Markdown
Contributor

Background

  • A client using the Go SDK to forward event data to mParticle asked whether mobile number is supported. UserIdentities was missing mobile_number (and the related phone identity fields), even though those identities already existed in IdentityType and in other SDKs such as the Apple SDK.

What Has Changed

  • Added MobileNumber, PhoneNumber2, and PhoneNumber3 to UserIdentities so events can include those identity values.
  • Added a unit test covering JSON marshaling for the new phone number identity fields.

Checklist

  • I have performed a self-review of my own code.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have tested this locally.

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

@BrandonStalnaker BrandonStalnaker self-assigned this Aug 5, 2026
@cursor

cursor Bot commented Aug 5, 2026

Copy link
Copy Markdown

PR Summary

Cursor Bugbot is generating a summary for commit 06b9ec6. Configure here.

@BrandonStalnaker
BrandonStalnaker force-pushed the feat/Add-Missing-User-Identities branch from 06b9ec6 to b185df0 Compare August 5, 2026 15:45
@BrandonStalnaker
BrandonStalnaker merged commit d19856f into master Aug 6, 2026
3 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