docs(push): release-dependent push open updates - #20102
Closed
turnipdabeets wants to merge 1 commit into
Closed
Conversation
Contributor
Deploy preview
Changed pages
|
Contributor
|
Vale prose linter → found 0 errors, 9 warnings, 3 suggestions in your markdown Full report → Copy the linter results into an LLM to batch-fix issues. Linter being weird? Update the rules!
|
| Line | Severity | Message | Rule |
|---|---|---|---|
| 12:132 | suggestion | Address the reader directly. Use 'you' instead of 'the user'. | PostHogDocs.DirectAddress |
| 32:110 | warning | 'APNs' is a possible misspelling. | PostHogBase.Spelling |
| 32:252 | warning | 'classpath' is a possible misspelling. | PostHogBase.Spelling |
| 32:382 | suggestion | Address the reader directly. Use 'you' instead of 'the user'. | PostHogDocs.DirectAddress |
| 59:54 | warning | 'APNs' is a possible misspelling. | PostHogBase.Spelling |
| 59:83 | warning | Use 'ID' instead of 'id'. | Vale.Terms |
| 61:1 | warning | 'Unregister' is a possible misspelling. | PostHogBase.Spelling |
| 69:18 | warning | 'unregistration' is a possible misspelling. | PostHogBase.Spelling |
| 116:112 | suggestion | Address the reader directly. Use 'you' instead of 'the user'. | PostHogDocs.DirectAddress |
| 116:203 | warning | 'classpath' is a possible misspelling. | PostHogBase.Spelling |
| 117:77 | warning | 'APNs' is a possible misspelling. | PostHogBase.Spelling |
| 117:105 | warning | Use 'ID' instead of 'id'. | Vale.Terms |
Contributor
Bundle reportTotal JS (gzip)8.78 MiB (+0.4 KiB / +0.0%) Eager graph (modules shipped in each entrypoint's initial chunks)
Largest modules in the
|
| Module | Size |
|---|---|
./src/data/mcp-tools.json |
1131.9 KiB |
css ./node_modules/.pnpm/css-loader@5.2.7_webpack@5.101.3/node_modules/css-loader/dist/cjs.js??ruleSet[1].rules[8].oneOf[1].use[1]!./node_modules/.pnpm/postcss-loader@4.3.0_postcss@8.5.6_webpack@5.101.3/node_modules/postcss-loader/dist/cjs.js??ruleSet[1].rules[8].oneOf[1].use[2]!./src/styles/global.css |
773.9 KiB |
./src/components/Stickers/Stickers.tsx |
696.4 KiB |
./node_modules/.pnpm/@radix-ui+react-icons@1.3.2_react@18.3.1/node_modules/@radix-ui/react-icons/dist/react-icons.esm.js |
481.4 KiB |
./node_modules/.pnpm/@posthog+brand@0.8.0_react@18.3.1/node_modules/@posthog/brand/dist/generated/hoggies/svg/x-ray.mjs |
480.8 KiB |
./node_modules/.pnpm/rehype-raw@7.0.0/node_modules/rehype-raw/lib/index.js + 29 modules |
395.1 KiB |
./node_modules/.pnpm/@posthog+brand@0.8.0_react@18.3.1/node_modules/@posthog/brand/dist/generated/hoggies/svg/im-the-driver.mjs |
385.7 KiB |
./src/hooks/useCustomers.tsx + 55 modules |
370.0 KiB |
./node_modules/.pnpm/@posthog+icons@0.36.6_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/@posthog/icons/dist/posthog-icons.es.js |
354.8 KiB |
./node_modules/.pnpm/react-markdown@8.0.7_@types+react@16.14.66_react@18.3.1/node_modules/react-markdown/lib/react-markdown.js + 88 modules |
351.4 KiB |
./src/components/ProductComparisonTable/index.tsx + 126 modules |
305.7 KiB |
./node_modules/.pnpm/cloudinary-core@2.14.0_lodash@4.17.21/node_modules/cloudinary-core/cloudinary-core.js |
281.9 KiB |
./node_modules/.pnpm/@posthog+brand@0.8.0_react@18.3.1/node_modules/@posthog/brand/dist/generated/hoggies/svg/doll-house.mjs |
281.7 KiB |
./node_modules/.pnpm/@posthog+brand@0.8.0_react@18.3.1/node_modules/@posthog/brand/dist/generated/hoggies/svg/director.mjs |
275.6 KiB |
./src/components/SearchUI/index.tsx + 87 modules |
273.7 KiB |
Eager-graph budgets are report-only until a baseline is established. Sizes are gzip of public/**/*.js; eager size is webpack module source bytes for the modules actually shipped in the entrypoint's initial chunks (post-tree-shake).
turnipdabeets
marked this pull request as draft
September 11, 2026 13:57
This was referenced Sep 11, 2026
Open
Release-dependent half of the push open docs update. The corrections that are true today moved to #20114.
turnipdabeets
force-pushed
the
docs/rn-push-warm-start
branch
from
September 12, 2026 16:08
19a1c55 to
694ea2f
Compare
Contributor
Author
|
Consolidated into #20114, which now carries every push notification docs change for this effort (this PR's dedupe wording included) and stays a draft until the SDK releases are out. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
Draft: every remaining item here waits on an SDK release. Do not merge before the release it names is published, since posthog.com deploys on merge.
The corrections that are true today, independent of any pending release, moved to #20114 (ready for review): the React Native iOS cold-launch gap, the iOS 14 floor on all three iOS surfaces, the Android warm-start coverage plus the killed-process gap, the notification-delegate callout, the missing React Native troubleshooting row, and the stale
posthog-android 3.62.0floor on the Flutter page.What is left on this branch:
react-native.mdx– "Capturing opens"Replaces "Only call it for opens automatic capture can't see itself, or the tap is counted twice" with an explicit "don't wire this to
messaging().onNotificationOpenedApp/getInitialNotification()" warning, and tells anyone who followed the old advice to remove their handler.This branch is based on
masterbefore #20114, so it still carries the snippet swap #20114 also makes. Rebase it once #20114 lands and the hunk reduces to the wording change.Still to write here, and what each waits on
android.mdx: "PostHog.capturePushNotificationOpenedbelow isn't deduplicated."invocation_id/action_id) within 5 minutes is skipped, first report wins. Pushes with no PostHog payload are still never deduplicated.mainalready released 3.64.0 with an unrelated minor, so #783 ships as 3.65.0, not 3.64.0.react-native.mdx: the "isn't deduplicated" wording on this branch@posthog/react-native-pluginrelease bumping both pins:com.posthog:posthog-android3.63.1 → 3.65.0 (PostHog/posthog-js#4919) andposthog_ios_version3.73.3 → 3.75.0.flutter.mdx: "the manual call isn't deduplicated against them, so the open is counted twice."posthog_flutterrelease whose floors clear both fixes. 5.41.0 declaresposthog-android [3.64.0,4.0.0)andPostHog >= 3.74.0, so it would resolve the fixes but does not require them – PostHog/posthog-flutter#578 raises the floors.react-native.mdx: iOS open coverage@posthog/react-native-pluginrelease with PostHog/posthog-js#4921.mainhas noprewarmPushNotificationOpenCapture()call today.react-native.mdx: "Opting out"capturePushNotificationOpened: falsealone no longer disables it;com.posthog.posthog.CAPTURE_PUSH_NOTIFICATION_OPENED=falseinInfo.plist(Expo:ios.infoPlist) is also required.@posthog/react-native-pluginrelease with PostHog/posthog-js#4921.ios.mdxtroubleshooting: "In a cross-platform host … callPostHogSDK.prewarmPushNotificationOpenCapture()… The PostHog Flutter plugin already does this for you."@posthog/react-native-pluginrelease with PostHog/posthog-js#4921.react-native.mdx: killed-process guidanceMainActivityby default,patchMainActivityNewIntent: falseopts out, and a bare React Native app that never runsexpo prebuildmust addsetIntent(intent)beforesuper.onNewIntent(intent)by hand. #20114 documents the gap; this documents the fix.posthog-react-native/ the plugin with PostHog/posthog-js#4929 (draft).ios.mdx: "Capturing opens"Checklist
vercel.json(no pages moved)Related PRs
One push-open capture effort across the mobile SDKs: count every notification tap exactly once, and stop losing taps the SDK starts too late to see.
firebase_messagingMerge order: posthog-android#783 and posthog-ios#828 first, then their releases. #4921, #4929 and #579 are independent and can go any time. #4919 and #578 go green once posthog-android 3.65.0 is published. Docs: #20114 can go now; #20102 last, after the releases.
Earlier work this builds on: PostHog/posthog-android#753, PostHog/posthog-ios#792, PostHog/posthog-js#4858, PostHog/posthog-flutter#556, PostHog/posthog-flutter#557, #19905.