Skip to content

Fix v1 introduction links#9

Open
velddev wants to merge 1 commit into
mainfrom
codex/fix-v1-introduction-links
Open

Fix v1 introduction links#9
velddev wants to merge 1 commit into
mainfrom
codex/fix-v1-introduction-links

Conversation

@velddev
Copy link
Copy Markdown
Contributor

@velddev velddev commented May 15, 2026

Summary

  • update v1 introduction links to use the canonical docs URL
  • fixes the three links reported by mint broken-links

Tests

  • npx mint broken-links

@mintlify
Copy link
Copy Markdown

mintlify Bot commented May 15, 2026

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

Project Status Preview Updated (UTC)
topgg 🟢 Ready View Preview May 15, 2026, 12:01 AM

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

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request replaces relative internal documentation links with absolute URLs pointing to the production domain. The reviewer recommends reverting these changes in favor of relative paths, as absolute URLs break local and staging preview environments and create inconsistencies with other links in the project.

Comment thread api/v0/introduction.mdx

<Warning>
The v0 API is the legacy version of the Top.gg API. **New integrations should use the [v1 API](/api/v1/introduction)** instead. The v0 API may not receive new features and could be deprecated in a future release.
The v0 API is the legacy version of the Top.gg API. **New integrations should use the [v1 API](https://docs.top.gg/api/v1/introduction)** instead. The v0 API may not receive new features and could be deprecated in a future release.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Using absolute URLs for internal documentation links is generally discouraged as it breaks local previews and staging environments (e.g., PR previews on Vercel or Netlify). Using a relative path like ../v1/introduction ensures the link remains environment-agnostic and consistent across different deployments.

  The v0 API is the legacy version of the Top.gg API. **New integrations should use the [v1 API](../v1/introduction)** instead. The v0 API may not receive new features and could be deprecated in a future release.

Comment thread api/v0/introduction.mdx

<Note>
Do not include a `Bearer` prefix. v0 tokens are passed as a raw string. Bearer-prefixed tokens are only valid on [v1 endpoints](/api/v1/introduction).
Do not include a `Bearer` prefix. v0 tokens are passed as a raw string. Bearer-prefixed tokens are only valid on [v1 endpoints](https://docs.top.gg/api/v1/introduction).
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Using absolute URLs for internal links breaks environment-agnostic navigation. Consider using a relative path instead to support local development and preview environments.

  Do not include a `Bearer` prefix. v0 tokens are passed as a raw string. Bearer-prefixed tokens are only valid on [v1 endpoints](../v1/introduction).

Comment thread index.mdx
Receive real-time vote notifications sent directly to your server.
</Card>
<Card title="API Reference" icon="code" href="/api/v1/introduction">
<Card title="API Reference" icon="code" href="https://docs.top.gg/api/v1/introduction">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Hardcoding the absolute production URL breaks preview environments and is inconsistent with other links in this file (e.g., the 'Webhooks' card on line 15). If the root-relative path was reported as broken, it might be worth checking if a trailing slash or a path relative to the root (e.g., api/v1/introduction) resolves the issue while keeping the link environment-agnostic.

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