docs: add Shopify performance guidance and fix the pixel snippet - #20013
Draft
posthog[bot] wants to merge 2 commits into
Draft
posthog[bot] wants to merge 2 commits into
posthog[bot] wants to merge 2 commits into
Conversation
The Shopify page had no performance section, so support had nothing to send store owners who ask about page weight. Its web pixel example also pasted a stale copy of the snippet, which loaded array.js from the ingestion host instead of the assets host and stubbed an outdated set of methods. Adds a "Reducing the impact on page speed" section, and replaces the pasted snippet with the shared snippet.mdx include so the page cannot drift again. Generated-By: PostHog Desktop Task-Id: 45b18981-f299-4535-bd63-478778e18d04
1 task
Contributor
Deploy preview
|
Contributor
|
Vale prose linter → found 1 errors, 9 warnings, 1 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 |
|---|---|---|---|
| 9:66 | warning | Use 'PostHog' instead of 'posthog'. | Vale.Terms |
| 9:104 | warning | Use 'Shopify' instead of 'shopify'. | Vale.Terms |
| 11:66 | warning | Use 'PostHog' instead of 'posthog'. | Vale.Terms |
| 11:104 | warning | Use 'Shopify' instead of 'shopify'. | Vale.Terms |
| 13:18 | warning | Avoid trivializing words. 'easy to' can sound dismissive to the reader. | PostHogDocs.Trivializers |
| 18:53 | warning | Use 'X' instead of 'x'. | Vale.Terms |
| 29:69 | error | Hi, Andy here... use an en dash ( – ) with spaces. On Mac, holding down the Option and hyphen key will give you an en dash. | PostHogBase.EnDash |
| 106:132 | warning | Use bold (text) for UI elements, not quotes. | PostHogDocs.UIBoldNotQuotes |
| 114:224 | suggestion | Address the reader directly. Use 'you' instead of 'the user'. | PostHogDocs.DirectAddress |
| 151:194 | warning | 'webpages' is a possible misspelling. | PostHogBase.Spelling |
| 178:12 | warning | 'regionality' is a possible misspelling. | PostHogBase.Spelling |
Contributor
Bundle reportTotal JS (gzip)8.85 MiB (+0.2 KiB / +0.0%) Eager graph (modules shipped in each entrypoint's initial chunks)
Largest modules in the
|
| Module | Size |
|---|---|
./src/data/mcp-tools.json |
1097.0 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 |
757.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 |
302.5 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.0 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).
Capitalizes the PostHog product names the new bullet links to, and names the two autocapture allowlist options instead of the plural noun the spelling rule does not recognize. Generated-By: PostHog Desktop Task-Id: 45b18981-f299-4535-bd63-478778e18d04
This was referenced Sep 11, 2026
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
Why: A paying agency running nine Shopify stores asked how to install PostHog with the lowest page-weight cost. This page had no performance section, and a search of the docs for Shopify performance guidance returned only blog posts, so support had nothing to send them.
posthog.initoptions that cut per-page work and what each one gives up.integrate/snippet.mdxinclude, and the example shows only theanalytics.subscribebody.The pasted copy had drifted badly. It loaded
array.jsfrom the ingestion host instead of-assets.i.posthog.com, and it stubbed a method list from several releases back, so a reader who copied it got a slower load and missing methods such ascaptureExceptionandsetPersonProperties. Using the include means it cannot drift again.disable_session_recordingrecorder.jsdownloaddisable_surveyssurveys.jsdownloadadvanced_disable_flagsautocapture: falsecapture()callscapture_heatmaps: falsecapture_performance: falseThe matching in-app onboarding step is added in PostHog/posthog#96878.
Every option name was checked against
contents/docs/libraries/js/config.mdxrather than recalled.prettier --checkpasses on the changed file.Note
The originating report stated that this example makes stores load the library twice on checkout. That could not be confirmed: Shopify runs custom pixels in a sandboxed frame, and the pasted snippet's own
__SVcheck makes a repeat call a no-op. The example is still wrong for the reasons above.Checklist
vercel.json— no page movedCreated with PostHog Desktop from this inbox report.