Skip to content

Document replay iframe embedding - #511

Merged
rgarcia merged 2 commits into
mainfrom
hypeship/replay-iframe-docs
Aug 20, 2026
Merged

Document replay iframe embedding#511
rgarcia merged 2 commits into
mainfrom
hypeship/replay-iframe-docs

Conversation

@rgarcia

@rgarcia rgarcia commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Description

Adds replay iframe guidance to the existing Replays page, including a responsive HTML example, viewport sizing, fullscreen support, URL handling, and CSP guidance.

Implementation Checklist

  • No sample app changes
  • No CLI changes

Testing

  • mintlify dev renders /browsers/replays
  • git diff --check

Visual Proof

Docs-only HTML example; no product UI changes.

Related Issue

None.


Note

Cursor Bugbot is generating a summary for commit eed24df. Configure here.

@mintlify

mintlify Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated (UTC)
Kernel 🟢 Ready View Preview Aug 20, 2026, 2:28 PM

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

cursor[bot]
cursor Bot approved these changes Aug 20, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Risk assessment: Very Low

Decision: Keep existing approval (risk did not increase)

Re-evaluated after the latest push (Specify replay iframe CSP). The PR is still a documentation-only change to one Mintlify page. The new commit replaces generic frame-src wording with an example CSP <meta> tag in the same HTML snippet. That does not add application, API, config, infrastructure, or auth changes, so risk remains Very Low. Approval is not dismissed and is not re-issued.

Factor Assessment
Codepaths modified One Mintlify docs page (browsers/replays.mdx, +27 / −0 vs main)
Blast radius Docs site copy only
Complexity New heading, prose, and a static HTML example
Infrastructure impact None
User-facing surface Public docs; no product UI or runtime behavior
Operational / security risk None in-repo. The snippet documents treating replay URLs as sensitive and example frame-src origins; it does not change product security.

CODEOWNERS: No CODEOWNERS file is present, so codeowner review is not required.

Approvals: This automation already approved the previous commit. GitHub reviewDecision is still APPROVED. No re-approval.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@rgarcia
rgarcia merged commit 96a6369 into main Aug 20, 2026
4 checks passed
@rgarcia
rgarcia deleted the hypeship/replay-iframe-docs branch August 20, 2026 14:46
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.

1 participant