Skip to content

Myronkaifung deprecate onboarding corereport guide#100

Open
myronkaifung wants to merge 15 commits into
mainfrom
myronkaifung-deprecate-onboarding-corereport-guide
Open

Myronkaifung deprecate onboarding corereport guide#100
myronkaifung wants to merge 15 commits into
mainfrom
myronkaifung-deprecate-onboarding-corereport-guide

Conversation

@myronkaifung

Copy link
Copy Markdown
Collaborator

No description provided.

@mintlify

mintlify Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
mixpanel-docs 🟢 Ready View Preview Jul 16, 2026, 10:12 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@mintlify

mintlify Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
mixpanel-docs 🟡 Building Jul 16, 2026, 10:08 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

Comment thread guides/guides-by-topic/core-reports.mdx
Comment thread guides/strategic-playbooks/onboarding-playbook.mdx
Comment thread guides/strategic-playbooks.mdx

@kurbycchua kurbycchua left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I left comments in the files, TLDR

  1. core-reports.mdx - should have some blurb referring to Mixpanel Fundamentals in Mixpanel University
  2. onboarding-playbook.mdx - same as above but retain / update "Methodology" section to give a preview before someone clicks into the Mixpanel Onboarding course in Mixpanel University
  3. We might need to tell readers that they need a Mixpanel account to access Mixpanel University?

@myronkaifung
myronkaifung marked this pull request as ready for review July 21, 2026 18:47
@myronkaifung
myronkaifung requested review from a team as code owners July 21, 2026 18:47
@myronkaifung
myronkaifung requested review from Tofufu and junhouse July 21, 2026 18:48
@greptile-apps

greptile-apps Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failures remain.

Important Files Changed

Filename Overview
docs.json Consolidates guide navigation and redirects removed child pages to their landing pages.
guides/guides-by-topic/core-reports.mdx Replaces individual report cards with a Mixpanel Fundamentals learning-path overview.
guides/strategic-playbooks/onboarding-playbook.mdx Replaces detailed onboarding child-page links with a concise high-level process and current resources.
guides/strategic-playbooks.mdx Adds a landing page for the remaining strategic playbooks.

Reviews (13): Last reviewed commit: "Merge branch 'main' into myronkaifung-de..." | Re-trigger Greptile

Comment thread docs.json Outdated
Comment thread guides/strategic-playbooks.mdx
Comment thread guides/strategic-playbooks/onboarding-playbook.mdx Outdated
Comment thread guides/strategic-playbooks/onboarding-playbook.mdx Outdated
Made some edits
1. Drop outdated links on Plan, Implement, and Launch
2. Edited Launch - since we change the content on that in MIxpanel Onboarding module in Mixpanel University
Added some more relavant links to text
Comment thread docs.json
Comment thread docs.json

@kurbycchua kurbycchua left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@myronkaifung perhaps need to update redirect links?

Updated per @amymadden-bit 's suggestion

Recommend going from "The Fundamentals module teaches you Mixpanel’s core reports and how to use them." to "The Fundamentals learning path teaches you about Mixpanel’s core reports and how to use them."
- To specify that it's a learning path, not just one module
- To add "about" to clarify the phrasing

Recommend going from "The module is hosted on Mixpanel University." to "The learning path is hosted on Mixpanel University."
- To link learning path, since it goes directly to the path and not the Mixpanel University homepage
-To specify that it's a learning path, not just one module
Updated per @amymadden-bit 's suggestion

Recommend going from "The Onboarding module walks you through implementing Mixpanel and getting the most from it." to "The Onboarding learning path walks you through implementing Mixpanel and getting the most from it."
- To specify that it's a learning path, not just one module

Recommend going from "The full module is hosted on Mixpanel University." to "The full learning path is hosted on Mixpanel University."
- To link learning path, since it goes directly to the path and not the Mixpanel University homepage
- To specify that it's a learning path, not just one module

Should we also add a sentence below the bullet point ("Drive a production rollout your team adopts") that says something like: "The rest of this page outlines that process at a high level, and the full learning path covers each step in more depth."
- To clarify that this is an overview of what the full path covers

## Guide to Product Analytics

<CardGroup cols={3}>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These doesn't seem to work in preview mode @myronkaifung

<Card title="Chatper 4: Retain Your Users" href="/guides/strategic-playbooks/guide-to-product-analytics/retain-your-users" />
<Card title="Chapter 5: Grow and Scale" href="/guides/strategic-playbooks/guide-to-product-analytics/grow-and-scale" />
<Card title="Conclusion" href="/guides/strategic-playbooks/guide-to-product-analytics/conclusion" />
</CardGroup>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These doesn't seem to work in preview mode @myronkaifung

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I think the preview deployment went stale. Can you check again?

@kurbycchua kurbycchua left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Cards for Strategic Playbooks doesn't seem to be working when clicking Guide to Product Analytics or Product Analytics Strategy it just opens up and expands the sub-items @myronkaifung

@greptile-apps

greptile-apps Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Want your agent to iterate on Greptile's feedback? Try greploops.

kurbycchua and others added 2 commits July 23, 2026 09:20
added "root": "guides/strategic-playbooks", under the Strategic Playbooks group to enable cards

@kurbycchua kurbycchua left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approving this PR

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