Skip to content

feat: SQDSDKS-7426 - Add RoktEvent mapping and emission#37

Merged
Mansi-mParticle merged 3 commits into
developmentfrom
feat/SQDSDKS-7425-Emit-Rokt-events
Jun 20, 2025
Merged

feat: SQDSDKS-7426 - Add RoktEvent mapping and emission#37
Mansi-mParticle merged 3 commits into
developmentfrom
feat/SQDSDKS-7425-Emit-Rokt-events

Conversation

@jamesnrokt

@jamesnrokt jamesnrokt commented Jun 19, 2025

Copy link
Copy Markdown
Collaborator

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

  • Add RoktEvent mapping and emission

Testing Plan

  • Was this tested locally? If not, explain why.
  • Tested locally with sample application and verified events were received
  • Added unit tests to validate changes

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

@BrandonStalnaker BrandonStalnaker changed the base branch from main to development June 19, 2025 12:41
@BrandonStalnaker BrandonStalnaker changed the base branch from development to main June 19, 2025 12:42
@Mansi-mParticle Mansi-mParticle changed the base branch from main to development June 19, 2025 15:32
@rmi22186 rmi22186 requested a review from Mansi-mParticle June 20, 2025 02:23
@jamesnrokt jamesnrokt changed the title feat: SQDSDKS-7425 - Add RoktEvent mapping and emission feat: SQDSDKS-7426 - Add RoktEvent mapping and emission Jun 20, 2025

@Mansi-mParticle Mansi-mParticle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tested with the Higgs sample app and LGTM

@Mansi-mParticle Mansi-mParticle merged commit 7c309dc into development Jun 20, 2025
28 of 32 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