Skip to content

ly_addGOver2 - #339

Open
lykoh01 wants to merge 1 commit into
mainfrom
linyee-koh/draft-oct-20
Open

lykoh01 wants to merge 1 commit into
mainfrom
linyee-koh/draft-oct-20

Conversation

@lykoh01

@lykoh01 lykoh01 commented Oct 20, 2026 •

Copy link
Copy Markdown

Mintlify

0 threads from 0 users in Mintlify

  • No unresolved comments

Review in Mintlify

- Updated docs/tracking-methods/sdks/go.mdx

Mintlify-Source: dashboard-editor
@mintlify

mintlify Bot commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated
mixpanel-docs 🟢 Ready View Preview Oct 20, 2026, 6:21 AM

@greptile-apps

greptile-apps Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 4/5

[Low impact] The change reaches only the Go SDK installation instructions in this guide.

The PR appears safe to merge, with a non-blocking fix needed to keep API links consistent.

Findings

  1. P2 API links target another version ▶

Reviews (1) · Last reviewed commit: "Updated mintlify pages" · Reviewed by Greptile

```shell
# install mixpanel library using Go get
go get github.com/mixpanel/mixpanel-go
go get github.com/mixpanel/mixpanel-go/v2

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 API links target another version

The install command now selects /v2, but the page’s API reference links still point to github.com/mixpanel/mixpanel-go. Readers looking up options or methods are sent to a different major version than the one they installed. Update the API reference links to /v2 alongside the install command.

@lykoh01 lykoh01 changed the title Draft from Oct 20 ly_addGOver2 Oct 20, 2026

This branch was successfully deployed

1 active deployment
staging — a3f14e83 Deployed Oct 20, 2026 by mintlify[bot]
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.

1 participant