Skip to content

fix: Restore ProGuard rules to preserve Rokt class members#590

Merged
Mansi-mParticle merged 1 commit into
developmentfrom
fix/rokt-events-api-proguard
Jun 24, 2025
Merged

fix: Restore ProGuard rules to preserve Rokt class members#590
Mansi-mParticle merged 1 commit into
developmentfrom
fix/rokt-events-api-proguard

Conversation

@thomson-t

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

  • events API from Rokt class was getting removed by proguard. Keep the class members and annotations for kotlin metadata

Testing Plan

  • Was this tested locally? If not, explain why.
  • Verified the local release snapshot aar file
Screenshot 2025-06-24 at 10 35 49 am

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

Comment thread android-core/proguard.pro
@thomson-t thomson-t force-pushed the fix/rokt-events-api-proguard branch 2 times, most recently from 2976209 to eca3f9d Compare June 24, 2025 03:23
`events` API from Rokt class was getting removed by proguard.
Keep the class members and annotations for kotlin metadata

Co-authored-by: James Newman <james.newman@rokt.com>
Signed-off-by: Thomson Thomas <125323226+thomson-t@users.noreply.github.com>
@thomson-t thomson-t force-pushed the fix/rokt-events-api-proguard branch from eca3f9d to 3a941ec Compare June 24, 2025 03:37
@Mansi-mParticle Mansi-mParticle self-requested a review June 24, 2025 16:04

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

LGTM

@Mansi-mParticle Mansi-mParticle merged commit 219b1d0 into development Jun 24, 2025
31 of 36 checks passed
@Mansi-mParticle Mansi-mParticle deleted the fix/rokt-events-api-proguard branch June 24, 2025 16:05
mparticle-automation added a commit that referenced this pull request Jun 24, 2025
## [5.70.1](v5.70.0...v5.70.1) (2025-06-24)

### Bug Fixes

* Restore ProGuard rules to preserve Rokt class members ([#590](#590)) ([219b1d0](219b1d0))

### Updates & Maintenance

* Create CODEOWNERS ([#591](#591)) ([eee1e27](eee1e27))
@mparticle-automation

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 5.70.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants