Repository navigation
chore(react-native-plugin)!: require posthog-android 4.0 - #5476
turnipdabeets wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
PostHog Review · Standard · reviewed bfa07d1: 1 Must fix.
This is the only automatic review while this pull request is a draft. Comment @posthog review to review new changes sooner. Otherwise, a review of the new changes runs when you mark it ready for review.
See this review and its findings in PostHog Review. Comment @posthog review --deep for a Deep review. A Deep review takes longer and looks deeper than a Standard review.
Coherence0 spec problems · 0 failing chokepoints · 2 spec gaps · 0 guard failures 44 components, 7 bullets (5 invariants, 2 requirements), 0 problems Chokepoints: none declared. Spec gaps: 2 entrances with outside or unknown trust and no traced control, and 10 more held by the adoption baseline
Close one with |
Warning
Prep PR: CI is expected to fail until posthog-android 4.0 is released.
com.posthog:posthog-android:4.0.0isn't on Maven Central yet. Merge into #5194 once 4.0 is published and CI is green. This is the Android side of #5237 (posthog-ios 4.0).Problem
#5194 (plugin 3.0) keeps posthog-android at 3.71.6. posthog-android 4.0 (PostHog/posthog-android#871) is coming. If plugin 3.0 is going to pick it up, it has to happen in this major.
Changes
packages/react-native-plugin/android/build.gradlenow depends oncom.posthog:posthog-android:4.0.0.chore/v4-remove-deprecated) API dump:PostHogAndroidConfig(apiKey, host)passes the token by position. The 4.0 constructor's first parameter isprojectToken, so the call still works.PostHogPreferences, includingANONYMOUS_ID/DISTINCT_ID, andPostHogSessionManager) stay public. chore!: make SDK-internal helpers non-public posthog-android#872 keeps them public for the wrappers.remoteConfig,debouncerDelayMs,propertiesSanitizer,getFeatureFlagPayload,PostHogOnFeatureFlags).minSdk23 and the same Kotlin settings.posthog-android-surveys-compose) that the plugin doesn't depend on.Before merging
4.0.x./i/v1/analytics/events, so reverse proxies have to forward that path. Add a patch changeset for it if so.Release info Sub-libraries affected
Libraries affected
Checklist
If releasing new changes
pnpm changesetto generate a changeset file🤖 Agent context
Autonomy: Human-driven (agent-assisted)
Made with Claude Code. I checked compatibility statically against the posthog-android 4.0 branch's API dump and source. Not tested: building the plugin against a local 4.0 build, or running the app.
🤖 Generated with Claude Code