Skip to content

Conversation

@thomson-t
Copy link
Contributor

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

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

@rmi22186 rmi22186 left a comment

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.

3 participants