Skip to content

Move Cloud tab docs under enterprise#451

Open
enyst wants to merge 3 commits intomainfrom
openhands-workspace-1926lVVI1FpsfBA2u2GZFh
Open

Move Cloud tab docs under enterprise#451
enyst wants to merge 3 commits intomainfrom
openhands-workspace-1926lVVI1FpsfBA2u2GZFh

Conversation

@enyst
Copy link
Copy Markdown
Collaborator

@enyst enyst commented Apr 11, 2026

  • I have read and reviewed the documentation changes to the best of my ability.
  • If the change is significant, I have run the documentation site locally and confirmed it renders as expected.

Summary of changes

  • move Cloud-tab-owned docs from openhands/usage/... to enterprise/usage/...
  • update the Cloud tab in docs.json to use the new enterprise/... paths
  • add redirects from the legacy openhands/usage/... routes so the visible docs URLs continue to work
  • update AGENTS.md to reflect the new Cloud docs location convention

This PR was created by an AI assistant (OpenHands) on behalf of the user.

Co-authored-by: openhands <openhands@all-hands.dev>
@enyst enyst requested a review from mamoodi as a code owner April 11, 2026 22:01
@mintlify
Copy link
Copy Markdown

mintlify bot commented Apr 11, 2026

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

Project Status Preview Updated (UTC)
all-hands-ai 🟢 Ready View Preview Apr 11, 2026, 10:02 PM

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

Copy link
Copy Markdown
Contributor

@all-hands-bot all-hands-bot left a comment

Choose a reason for hiding this comment

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

🔴 Needs rework - The file reorganization structure is sound, but you've moved the files without updating their internal cross-references.

[CRITICAL ISSUES]

  • Internal Links (52 broken): All moved files still reference /openhands/usage/ paths internally. Examples:

    • enterprise/usage/settings/api-keys-settings.mdx line 7: /openhands/usage/cloud/openhands-cloud
    • enterprise/usage/settings/integrations-settings.mdx line 10: /openhands/usage/cloud/openhands-cloud
    • Total: 34 refs to /openhands/usage/cloud/, 18 to other moved paths
  • Duplicate Navigation: integrations-settings appears in both Settings and Integrations groups (docs.json lines 390 and 397)

[FIX]

Run this search-and-replace across all moved files:

find enterprise/usage -name '*.mdx' -type f -exec sed -i \
  's|/openhands/usage/cloud/|/enterprise/usage/cloud/|g; \
   s|/openhands/usage/settings/|/enterprise/usage/settings/|g; \
   s|/openhands/usage/automations/|/enterprise/usage/automations/|g; \
   s|/openhands/usage/api/|/enterprise/usage/api/|g' {} +

Remove the duplicate integrations-settings entry from the Integrations group in docs.json.

VERDICT:Needs rework - Must fix broken internal links

KEY INSIGHT: When moving documentation, internal cross-references matter as much as the file paths.

Co-authored-by: openhands <openhands@all-hands.dev>
Copy link
Copy Markdown
Collaborator Author

enyst commented Apr 11, 2026

Addressed the review feedback in 8332c17:

  • updated internal links inside the moved enterprise/usage/... pages so links to other moved pages now point at their new enterprise/... locations
  • left links to unmoved pages (for example application-settings, llm-settings, mcp-settings, and secrets-settings) unchanged
  • removed the duplicate integrations-settings entry from the Cloud tab
  • re-verified that there are no remaining references from enterprise/usage/... docs to the old moved paths, and that the Cloud tab no longer has duplicate page entries

This comment was created by an AI assistant (OpenHands) on behalf of the user.

Copy link
Copy Markdown
Contributor

@all-hands-bot all-hands-bot left a comment

Choose a reason for hiding this comment

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

🟡 Acceptable - The file reorganization is technically sound, but lacks build verification.

KEY INSIGHT: This is a straightforward directory restructure with proper link updates and backward-compatible redirects. The execution is pragmatic and clean, but you didn't prove it works.

Copy link
Copy Markdown
Contributor

@all-hands-bot all-hands-bot left a comment

Choose a reason for hiding this comment

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

🟡 Acceptable - Files moved correctly with proper internal updates, but external references weren't updated.

KEY INSIGHT: The reorganization is half-complete—moved files are clean, but 12 other docs still use old paths, creating redirect hops and inconsistency.

Co-authored-by: openhands <openhands@all-hands.dev>
Copy link
Copy Markdown
Contributor

@all-hands-bot all-hands-bot left a comment

Choose a reason for hiding this comment

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

🟡 Acceptable - Clean directory reorganization with proper redirects and comprehensive link updates.

KEY INSIGHT: This is a well-executed mechanical refactor. You mentioned running mint broken-links in a review comment, but the PR description checkbox for local verification is unchecked—consider documenting the verification in an Evidence section for future reference.

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