Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Update to Partner Latest version 7.0.2 #100

Merged

Conversation

Mansi-mParticle
Copy link
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

  • Update Clevertap kit to version 7.0.2 (latest version)
  • Update cleverTapAttributionIdentifier method to getCleverTapID

Testing Plan

  • Was this tested locally? If not, explain why.
  • Ran unit tests and verified functionality with sample application

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

Copy link

@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!

@rmi22186
Copy link

rmi22186 commented Oct 11, 2024

good call on updating the deprecated method in favor of getCleverTapID, which looks like it happened in 4.2!

Copy link

@mmustafa-tse mmustafa-tse left a comment

Choose a reason for hiding this comment

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

LGTM too!

@Mansi-mParticle Mansi-mParticle merged commit 29ecb70 into development Oct 15, 2024
16 checks passed
@Mansi-mParticle Mansi-mParticle deleted the feat/SQDSDKS-5878-update-to-version-7.0.2 branch October 15, 2024 14:32
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