Skip to content

docs: align Flutter native dependency versions for CNAME updates#73

Merged
denischilik merged 2 commits into
mainfrom
feat/flutter-rokt-cname-5-79
May 19, 2026
Merged

docs: align Flutter native dependency versions for CNAME updates#73
denischilik merged 2 commits into
mainfrom
feat/flutter-rokt-cname-5-79

Conversation

@denischilik
Copy link
Copy Markdown
Contributor

Summary

  • pin Android native dependencies to com.mparticle:android-core:5.79.0 and com.mparticle:android-rokt-kit:5.79.0 in plugin/example Gradle configs and setup docs
  • bump iOS podspec dependency guidance to mParticle-Apple-SDK ~> 9.2
  • add native CNAME initialization examples in README for Android (NetworkOptions.withNetworkOptions) and iOS (MPNetworkOptions.customBaseURL)

Test plan

  • Documentation-only verification: review README.md, CHANGELOG.md, and ios/mparticle_flutter_sdk.podspec for version consistency
  • Android build smoke check in example app with pinned dependency versions
  • iOS pod install smoke check with mParticle-Apple-SDK ~> 9.2

Pin Android mParticle/Rokt dependencies to 5.79.0 and update iOS pod guidance to 9.2 while documenting native CNAME configuration examples for both platforms.
@cursor
Copy link
Copy Markdown

cursor Bot commented May 14, 2026

PR Summary

Medium Risk
Medium risk because it changes shipped native dependency versions (Android Gradle and iOS podspec), which can affect build compatibility and runtime behavior even though no Dart/plugin logic changes.

Overview
Updates native dependency guidance and defaults to match newer SDK versions and support CNAME routing.

Pins Android dependencies to com.mparticle:android-core:5.79.0 (and android-rokt-kit:5.79.0 where applicable) in the README and in the plugin/example Gradle builds, and bumps the iOS CocoaPods dependency in mparticle_flutter_sdk.podspec/docs from mParticle-Apple-SDK ~> 9.1 to ~> 9.2.

Adds optional CNAME initialization examples: Android via NetworkOptions.withNetworkOptions(...) and iOS via MPNetworkOptions.customBaseURL, and records these changes in CHANGELOG.md.

Reviewed by Cursor Bugbot for commit 4faeef9. Bugbot is set up for automated code reviews on this repo. Configure here.

@denischilik denischilik requested a review from thomson-t May 14, 2026 18:59
Comment thread README.md Outdated
Keep Android and iOS initialization snippets minimal by removing custom CNAME setup from the default path, and document CNAME configuration in separate optional examples.
@denischilik denischilik requested a review from thomson-t May 18, 2026 13:41
Copy link
Copy Markdown
Contributor

@BrandonStalnaker BrandonStalnaker left a comment

Choose a reason for hiding this comment

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

I'm somewhat hesitant to pin to specific versions rather than to next major but the codes good in general and I think that decision just boils down to personal opinion

@denischilik denischilik merged commit ae8f33e into main May 19, 2026
6 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