Skip to content

docs: add enterprise directory PR restriction notice#453

Merged
jpelletier1 merged 3 commits intomainfrom
docs/enterprise-pr-restriction
Apr 13, 2026
Merged

docs: add enterprise directory PR restriction notice#453
jpelletier1 merged 3 commits intomainfrom
docs/enterprise-pr-restriction

Conversation

@jpelletier1
Copy link
Copy Markdown
Contributor

  • 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

Adds a warning notice under the "Pull Request Process" section of the contributing page explaining that:

  1. Pull requests for changes in the enterprise/ directory of the OpenHands/OpenHands repository cannot be accepted at this time, as this part of the codebase is commercially licensed
  2. Users are encouraged to provide feedback for OpenHands Enterprise by creating an issue in the OpenHands repository instead

The warning includes links to:


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

@jpelletier1 can click here to continue refining the PR

openhands-agent and others added 2 commits April 13, 2026 15:32
@jpelletier1
Copy link
Copy Markdown
Contributor Author

@OpenHands the mintlify preview seems broken. Can you investigate why?

@openhands-ai
Copy link
Copy Markdown

openhands-ai bot commented Apr 13, 2026

I'm on it! jpelletier1 can track my progress at all-hands.dev

Copy link
Copy Markdown
Contributor Author

I investigated the Mintlify preview and found that the preview is actually working correctly for the changed file.

What I found:

  • ✅ The contributing page (/overview/contributing) renders correctly with the Warning component properly displayed
  • ✅ All PR checks pass (including Mintlify Deployment, Mintlify Validation, and broken-links check)
  • ✅ Running mintlify broken-links locally confirms no broken links

About the 404 pages:
The 404 errors for other pages (like /enterprise/index, /overview/introduction, etc.) appear to be a limitation of Mintlify's PR preview deployment - PR previews may only build/serve the changed pages rather than the full site. This is expected behavior and not an error in the PR.

Verification:

To verify the full site locally, you can run:

npx mintlify dev

The PR changes are ready for review. Is there anything specific about the preview that looked broken to you?

@openhands-ai
Copy link
Copy Markdown

openhands-ai bot commented Apr 13, 2026

Final Summary

I investigated the Mintlify preview issue for PR #453 as requested.

Investigation Results

The preview is NOT broken - it's working as expected for Mintlify PR previews.

Findings:

  • ✅ The changed page (/overview/contributing) renders correctly with the Warning component displayed properly
  • ✅ All PR checks pass: Mintlify Deployment (success), Mintlify Validation (success), broken-links check (success)
  • ✅ Local mintlify broken-links check confirms no broken links
  • ✅ The /enterprise/index link in the Warning is valid and works on production

Root Cause of 404s:
The 404 errors on other pages (e.g., /enterprise/index, /overview/introduction, /sdk) are a limitation of Mintlify's PR preview deployments, which only build/serve the changed files rather than the entire site. This is expected behavior, not a bug.

Action Taken

I posted a comment on PR #453 explaining the investigation results and asking if there was something specific that looked broken to the user.

No Code Changes Made

This was purely an investigation task - no code modifications were needed because the PR changes are correct and functioning as intended.

@mintlify
Copy link
Copy Markdown

mintlify bot commented Apr 13, 2026

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

Project Status Preview Updated (UTC)
all-hands-ai 🟢 Ready View Preview Apr 13, 2026, 3:46 PM

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

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

Update: Found the root cause

I've identified this is a known Mintlify bug: Preview deployments show 404s if branch name was used in prior deployments.

The issue:

When a branch name (like docs/enterprise-pr-restriction) has been used in a previous deployment, new deployments to that same branch name can result in all pages showing 404 except for the sidebar navigation.

Symptoms we're seeing:

  • /overview/contributing works (200)
  • /overview/introduction returns 404
  • /enterprise returns 404
  • ❌ All other pages return 404

Workaround:

A manual redeployment from the Mintlify Dashboard should fix this. Someone with access to the Mintlify dashboard can:

  1. Go to the Previews section
  2. Find this PR's preview
  3. Click the "Deploy" button to trigger a manual redeploy

Note:

This is a Mintlify-side issue, not a problem with the PR changes. The documentation changes are correct - you can verify by running npx mintlify dev locally.

@jpelletier1
Copy link
Copy Markdown
Contributor Author

Mintlify preview now working. Here's a preview of the change: https://allhandsai-docs-enterprise-pr-restriction.mintlify.app/overview/contributing#pull-request-process

@jpelletier1 jpelletier1 marked this pull request as ready for review April 13, 2026 16:09
@jpelletier1 jpelletier1 requested review from neubig and rbren April 13, 2026 16:09
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.

🟢 Good taste - Clear documentation of contribution boundaries. Solves a real problem (users need to know where they can contribute) with straightforward language. One minor suggestion below about temporal weasel words.

Comment thread overview/contributing.mdx
@jpelletier1 jpelletier1 merged commit 82b5c7a into main Apr 13, 2026
8 checks passed
@jpelletier1 jpelletier1 deleted the docs/enterprise-pr-restriction branch April 13, 2026 16:22
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.

4 participants