Skip to content

PLTF-2503: add Laminar analytics setup guide#503

Open
aivong-openhands wants to merge 10 commits into
mainfrom
av/laminar-enterprise-docs
Open

PLTF-2503: add Laminar analytics setup guide#503
aivong-openhands wants to merge 10 commits into
mainfrom
av/laminar-enterprise-docs

Conversation

@aivong-openhands
Copy link
Copy Markdown
Contributor

@aivong-openhands aivong-openhands commented May 12, 2026

Summary

  • Adds enterprise/analytics.mdx, a step-by-step guide for enabling Laminar-based analytics in OpenHands Enterprise (config → deploy → project setup → ingest-only API key → verify traces).
  • Adds the 11 supporting screenshots under enterprise/images/.

Notes

  • A <video> tag references ./videos/laminar-demo.mp4, but the 189MB source file is intentionally not committed (exceeds GitHub's 100MB limit and would bloat the repo). Before marking ready-for-review, the video needs to be compressed and re-hosted (recommended: GitHub user-attachments URL, matching the convention used elsewhere in this repo).

Test plan

  • Run mint dev locally and confirm all images render
  • Compress the demo video and swap the src to a GitHub user-attachments URL
  • Confirm video plays in the local preview
  • Verify the page is reachable via the docs nav (check docs.json if needed)

Adds a step-by-step guide for enabling Laminar-based analytics in
OpenHands Enterprise, covering configuration, deployment, project
setup, ingest-only API keys, and verifying traces.
@mintlify
Copy link
Copy Markdown

mintlify Bot commented May 12, 2026

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

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

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

@aivong-openhands
Copy link
Copy Markdown
Contributor Author

laminar-demo-compressed.mp4

@aivong-openhands aivong-openhands changed the title docs(enterprise): add Laminar analytics setup guide PLTF-2503: add Laminar analytics setup guide May 12, 2026
@aivong-openhands aivong-openhands marked this pull request as ready for review May 12, 2026 16:42
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.

Documentation structure is clear and comprehensive. Two minor suggestions for improved user experience.


[RISK ASSESSMENT]

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

Documentation-only change with no code modifications. Follows established patterns, includes proper screenshots, and uses recommended GitHub user-attachments for video hosting. All referenced internal links exist.

Comment thread enterprise/analytics.mdx Outdated
Comment thread enterprise/analytics.mdx Outdated
aivong-openhands and others added 2 commits May 12, 2026 11:47
Co-authored-by: OpenHands Bot <contact@all-hands.dev>
Co-authored-by: OpenHands Bot <contact@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.

Documentation is well-structured and comprehensive. One minor style consistency suggestion.

Comment thread enterprise/analytics.mdx Outdated
Co-authored-by: OpenHands Bot <contact@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.

Documentation is comprehensive and well-structured. All previous review feedback has been addressed.

[RISK ASSESSMENT]

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

Documentation-only change with no code modifications. Step-by-step guide follows established enterprise documentation patterns, includes appropriate screenshots, and uses GitHub user-attachments for video hosting.

Copy link
Copy Markdown
Contributor

@juanmichelini juanmichelini left a comment

Choose a reason for hiding this comment

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

Suggested a small edit to highlight the importance of ingest api key.
I rather we have some redundancy than having an incident.

Comment thread enterprise/analytics.mdx Outdated
Co-authored-by: Juan Michelini <juan@juan.com.uy>
Copy link
Copy Markdown
Contributor

@juanmichelini juanmichelini left a comment

Choose a reason for hiding this comment

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

thanks, LGTM!

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