Skip to content

Conversation

@Hritwik001
Copy link

🧩 Summary
Fixes the broken "Components" link on the Internals → Overview page.

Old: ./components → 404
New: ./architecture/components → valid
Keeps consistency with other relative links (./security, ./service-tokens).

🔍 Rationale
The Components page actually resides under internals/architecture/components.
This update corrects the Overview link to match the current documentation structure.

🧪 Verification
Checked old URL → 404 ❌
Checked new URL → loads Components page ✅
Relative path syntax verified for Mintlify docs.

🧱 Type of Change
📝 Docs fix (non-breaking)

🔗 Related Issue
Closes #4660

Updated the broken "Components" link on Internals → Overview page.

- Old: ./components → 404
- New: ./architecture/components → valid
- Keeps consistency with other relative links on the page (./security, ./service-tokens)

Verified:
- Existing target (/docs/internals/components) does not exist.
- Correct page lives at /docs/internals/architecture/components.

Closes Infisical#4660
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Greptile Overview

Greptile Summary

This PR fixes a broken documentation link in the Internals Overview page. The "Components" link was incorrectly pointing to ./components, which resulted in a404error. The link has been updated to ./architecture/components to match the actual location of the Components documentation. This is a straightforward path correction that maintains consistency with other relative links on the same page (such as ./security and ./service-tokens), ensuring proper navigation within the Mintlify documentation framework.

Changed Files
Filename Score Overview
docs/internals/overview.mdx 5/5 Fixed broken documentation link from ./components to ./architecture/components

Confidence score: 5/5

  • This PR is extremely safe to merge with no risk of breaking functionality
  • Score reflects a simple, well-verified documentation path fix with clear testing and validation
  • No files require special attention - this is a single-line documentation link correction

Sequence Diagram

sequenceDiagram
    participant User
    participant "Documentation Page" as Docs
    participant "Internals Overview" as Overview
    participant "Components Page" as Components
    
    User->>Docs: "Navigate to Internals Overview"
    Docs->>Overview: "Load overview.mdx"
    Overview->>User: "Display page with links"
    User->>Overview: "Click 'Components' card"
    Overview->>Components: "Navigate to ./architecture/components"
    Components->>User: "Display Components page (200 OK)"
Loading

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@maidul98
Copy link
Collaborator

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@Hritwik001

This comment was marked as spam.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants