TOF-440: Split warehouse connectors into per-connector pages - #173
Open
tylergoerzen-mxp with Copilot wants to merge 7 commits into
Open
TOF-440: Split warehouse connectors into per-connector pages#173tylergoerzen-mxp with Copilot wants to merge 7 commits into
tylergoerzen-mxp with Copilot wants to merge 7 commits into
Conversation
Co-authored-by: tylergoerzen-mxp <259741734+tylergoerzen-mxp@users.noreply.github.com>
Contributor
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
Co-authored-by: tylergoerzen-mxp <259741734+tylergoerzen-mxp@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Split warehouse connectors into per-connector pages
TOF-440: Split warehouse connectors into per-connector pages
Aug 18, 2026
Contributor
Confidence Score: 4/5The PR is not yet safe to merge because the Postgres future-table grant still fails for tables created by roles other than the one executing the documented command. The Postgres instructions continue to promise future-table access without configuring default privileges for each actual table-owner role, leaving newly created tables unreadable by the connector. Files Needing Attention: docs/tracking-methods/warehouse-connectors/postgres.mdx
|
| Filename | Overview |
|---|---|
| docs.json | Replaces the single warehouse connector navigation entry with a rooted group containing five connector pages. |
| docs/tracking-methods/warehouse-connectors.mdx | Converts the monolithic connector tabs into an overview linking to dedicated pages while retaining shared concepts and FAQs. |
| docs/tracking-methods/warehouse-connectors/postgres.mdx | Adds dedicated Postgres setup, permissions, SSL, networking, and sync-support documentation. |
| docs/tracking-methods/warehouse-connectors/bigquery.mdx | Adds dedicated BigQuery connection and Mirror-mode documentation. |
| docs/tracking-methods/warehouse-connectors/snowflake.mdx | Adds dedicated Snowflake connection and Streams-based Mirror documentation. |
| docs/tracking-methods/warehouse-connectors/databricks.mdx | Adds dedicated Databricks authentication, compute, permissions, and Change Data Feed documentation. |
| docs/tracking-methods/warehouse-connectors/redshift.mdx | Adds dedicated Redshift setup and snapshot-based Mirror documentation. |
Reviews (6): Last reviewed commit: "Merge branch 'main' into copilot/tof-440..." | Re-trigger Greptile
- Re-add the Databricks setup video (Fr-isqO1w2s). The other three connector videos survived the split; this one was dropped. - Repeat the paid-add-on and admin/owner-role notes on each connector page. They only remained on the hub, so anyone landing on a connector page from search saw setup steps with no mention of gating. - Drop the duplicate nav entry: the group already links its root, so listing the overview in pages rendered it twice in the sidebar. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Rec #2 step 4 asked for both and the split carried neither over. - Add a commented, copy-pasteable SQL block creating a read-only role with CONNECT, USAGE, and SELECT, plus ALTER DEFAULT PRIVILEGES so tables added later still sync. Missing USAGE on the schema is the failure mode where the connector authenticates but sees no tables. - Replace the two-line SSL description with a table showing what verify-full and verify-ca each verify, and state the actual tradeoff: verify-ca skips the hostname check, so it accepts any server holding a certificate from a trusted CA. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Four defects in the setup examples, all of which fail on copy-paste: - The key-pair CREATE USER assigned DEFAULT_ROLE=MIXPANEL_IMPORT_ROLE, a role the block never creates. Only MIXPANEL_ROLE exists. - Comments used '#', which Snowflake does not support. Snowflake accepts '--' and '/* */', so "WAREHOUSE_SIZE = XSMALL # consider increasing" was a syntax error rather than a comment. - GRANT USAGE/MONITOR ON MIXPANEL_WAREHOUSE omitted the required WAREHOUSE object-type keyword. - The storage-integration grant used "TO MIXPANEL_ROLE" while every other grant on the page uses "TO ROLE"; made it consistent. Also tag the five SQL blocks as sql rather than jsx. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.
Splits
warehouse-connectors.mdx(918 lines, five connectors inside<Tab>components on one URL) into an overview plus one page per connector.Fixes: https://linear.app/mixpanel/issue/TOF-440/aeo-qw3-split-warehouse-connectors-into-per-connector-pages
Why this matters for AEO
The content was already deep and good. It just had nowhere to be cited from.
Answer engines retrieve passages but cite URLs. A query like "mixpanel postgres connector" or "connect snowflake to mixpanel" had no dedicated page to rank or be cited — the answer existed, buried in a tab on a URL titled "Warehouse Connectors." Tab content also sits behind a click, and no non-Google AI crawler executes JavaScript.
Two supporting reasons this one is worth doing:
/docs/data-pipelines/integrations/. Import connectors were the inconsistent half.Title-to-query similarity is the largest single citation predictor measured (0.602 for cited vs 0.484 for non-cited, Ahrefs 1.4M-prompt study). Five pages titled for their warehouse beat one titled for the category.
Structure
warehouse-connectors.mdx— overview,CardGroupto each connector, retains all shared content (Table Types, Sync Modes incl. the billing-cost column, Sync Frequency, FAQ, Billing FAQ)bigquery,snowflake,databricks,redshift,postgres— connection setup and Mirror behavior per warehouse (Postgres does not support Mirror)No URL was removed.
/docs/tracking-methods/warehouse-connectorsstill resolves, as the overview.Content restored and corrected during review
Fr-isqO1w2s.rootand also listed it inpages, rendering the overview twice.Broken SQL fixed on the Snowflake page
Four defects, each of which fails on copy-paste:
CREATE USER ... DEFAULT_ROLE=MIXPANEL_IMPORT_ROLEreferenced a role the block never creates (it createsMIXPANEL_ROLE)#, which Snowflake does not support — soWAREHOUSE_SIZE = XSMALL # consider increasingwas a syntax error, not a commentGRANT USAGE/MONITOR ON MIXPANEL_WAREHOUSEomitted the requiredWAREHOUSEobject-type keywordTO MIXPANEL_ROLEwhere every sibling usesTO ROLEBlocks are now tagged
sqlrather thanjsx.Postgres additions
Rec #2 asked for both and the original split carried neither:
CONNECT,USAGE,SELECT, plusALTER DEFAULT PRIVILEGESso tables added later still sync. MissingUSAGEon the schema is the failure mode where the connector authenticates but sees no tables.verify-caskips the hostname check, so it accepts any server holding a certificate from a trusted CA.Verification
Section-by-section accounting against the original: every heading, table row, warning, code sample, and image is present in exactly one new page. All 11 inbound anchors against the hub still resolve. No redirects needed — nothing was renamed.