Skip to content

feat: Add Rokt purchaseFinalized method#54

Merged
thomson-t merged 2 commits into
developmentfrom
feat/SQDSDKS-7431-purchase-finalized-api
Jun 24, 2025
Merged

feat: Add Rokt purchaseFinalized method#54
thomson-t merged 2 commits into
developmentfrom
feat/SQDSDKS-7431-purchase-finalized-api

Conversation

@thomson-t

@thomson-t thomson-t commented Jun 23, 2025

Copy link
Copy Markdown
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

  • Add purchaseFinalized method to the Flutter Rokt API
  • Implement native calls for purchaseFinalized on Android and iOS
  • Add unit tests to verify the new functionality
  • Add a test helper to clear placeholders for test isolation

Testing Plan

  • Was this tested locally? If not, explain why.
  • Tested the calls are invoking the kit

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

- Add `purchaseFinalized` method to the Flutter Rokt API
- Implement native calls for `purchaseFinalized` on Android and iOS
- Add unit tests to verify the new functionality
- Add a test helper to clear placeholders for test isolation

Signed-off-by: Thomson Thomas <thomson.thomas@rokt.com>
@thomson-t thomson-t force-pushed the feat/SQDSDKS-7431-purchase-finalized-api branch from e1ca3ad to 34d9c69 Compare June 24, 2025 04:31
@thomson-t thomson-t changed the title WIP feat: SQDSDKS-7431 - Add Rokt purchaseFinalized method feat: SQDSDKS-7431 - Add Rokt purchaseFinalized method Jun 24, 2025
@thomson-t thomson-t marked this pull request as ready for review June 24, 2025 04:33
@thomson-t thomson-t requested a review from a team June 24, 2025 04:34
@rmi22186 rmi22186 changed the title feat: SQDSDKS-7431 - Add Rokt purchaseFinalized method feat: Add Rokt purchaseFinalized method Jun 24, 2025

@rmi22186 rmi22186 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm!

@thomson-t thomson-t merged commit c1e8164 into development Jun 24, 2025
11 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