Skip to content

docs: add analytics to enterprise docs dns and cert requirements#523

Merged
aivong-openhands merged 1 commit into
mainfrom
av/add-analytics-domain-enterprise
May 26, 2026
Merged

docs: add analytics to enterprise docs dns and cert requirements#523
aivong-openhands merged 1 commit into
mainfrom
av/add-analytics-domain-enterprise

Conversation

@aivong-openhands
Copy link
Copy Markdown
Contributor

@aivong-openhands aivong-openhands commented May 26, 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

@aivong-openhands aivong-openhands marked this pull request as ready for review May 26, 2026 15:14
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 - Clean documentation update that correctly documents the analytics subdomain requirement.

Context: The analytics.app.<your-domain> subdomain hosts the Laminar UI for observability and tracing (as referenced in /enterprise/analytics.mdx). This change ensures users configure DNS and TLS certificates correctly during installation.

What's Good:

  • Follows the exact pattern of existing subdomain entries
  • Updates both the DNS table and the test script consistently
  • Properly alphabetized between app. and auth.app.
  • Covered by the TLS certificate SAN requirements ("for all of the above domains")

[RISK ASSESSMENT]

  • [Overall PR] ⚠️ Risk Assessment: 🟢 LOW

This is a documentation-only change that improves accuracy by documenting an existing product requirement. No code changes, no breaking changes, no security concerns. The change correctly reflects the infrastructure needs for the analytics feature.

VERDICT:
Worth merging: Documentation accurately reflects product requirements

KEY INSIGHT:
Complete and correct documentation of infrastructure requirements prevents deployment issues.


Was this automated review useful? React with 👍 or 👎 to this review to help us measure review quality.
Workflow run: https://github.com/OpenHands/docs/actions/runs/26457195508

@mintlify
Copy link
Copy Markdown

mintlify Bot commented May 26, 2026

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

Project Status Preview Updated (UTC)
all-hands-ai 🟢 Ready View Preview May 26, 2026, 3:20 PM

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

@aivong-openhands aivong-openhands merged commit db17ed7 into main May 26, 2026
8 of 9 checks passed
@aivong-openhands aivong-openhands deleted the av/add-analytics-domain-enterprise branch May 26, 2026 15:21
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