Skip to content

docs(react-native): Document opening the feedback form over a presented sheet on iOS - #20255

Merged
antonis merged 1 commit into
masterfrom
antonis/docs-rn-feedback-over-ios-sheet
Oct 27, 2026
Merged

antonis merged 1 commit into
masterfrom
antonis/docs-rn-feedback-over-ios-sheet

Conversation

@antonis

@antonis antonis commented Oct 27, 2026

Copy link
Copy Markdown
Contributor

DESCRIBE YOUR PR

Adds a troubleshooting subsection to the React Native User Feedback page covering a platform limitation on iOS: because UIKit allows only one presentation per view controller, the default Modal-based Feedback Form (showFeedbackForm(), shake-to-report, the auto-injected button) can't open while a native-stack screen is presented as a sheet (presentation: "formSheet" | "modal"). The form silently doesn't appear. Android is unaffected.

The new section documents the supported workaround: render the FeedbackForm component yourself and, on iOS, wrap it in react-native-screens' FullWindowOverlay so it draws above the presented sheet (with a Modal fallback on Android).

Context: getsentry/sentry-react-native#6875. A proper default-on fix (native own-window presentation) is tracked there as an enhancement; this documents the supported path in the meantime.

IS YOUR CHANGE URGENT?

  • Urgent deadline (GA date, etc.): YYYY-MM-DD
  • Other deadline: YYYY-MM-DD
  • No deadline: Not urgent, can wait up to 1 week+

PRE-MERGE CHECKLIST

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

🤖 Generated with Claude Code

…ed sheet on iOS

On iOS, UIKit allows one presentation per view controller, so the default
Modal-based feedback form can't open while a native-stack sheet is presented.
Document rendering the FeedbackForm component inside a FullWindowOverlay as the
supported workaround.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
sentry-docs Ready Ready Preview Oct 27, 2026 2:09pm UTC
1 Skipped Deployment
Project Deployment Actions Updated
develop-docs Ignored Ignored Preview Oct 27, 2026 2:09pm UTC

Request Review

@github-actions github-actions Bot added the Priority: Normal Docs review has no urgent deadline label Oct 27, 2026
@codeowner-assignment
codeowner-assignment Bot requested a review from a team October 27, 2026 13:59
@antonis
antonis marked this pull request as ready for review October 27, 2026 14:11
@sentry-junior

sentry-junior Bot commented Oct 27, 2026

Copy link
Copy Markdown
Contributor

Screenshots

Rendered from the Vercel preview for this branch (antonis/docs-rn-feedback-over-ios-sheet, commit 381c3fa). The "before" shot is the same page on production (docs.sentry.io). Highlights were added for review only.

React Native User Feedback (/platforms/react-native/user-feedback/)

Before: the offline note goes straight into Session Replay
Before: React Native user feedback

After: new "Opening the Feedback Form Over a Presented Sheet on iOS" subsection with the UIKit limitation, the FullWindowOverlay example, and the react-native-screens note
After: Feedback Form over a presented sheet on iOS

The FeedbackForm link in the new subsection resolves to the existing anchor on the page.

@cursor

cursor Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

The plan checks that the new heading is on the published page. It also checks page loads, path 404s, and client errors. An issue escalates when the heading is missing or those failure checks rise.

Monitoring in production

Services: sentry-docs.

Mention @change-monitor in a comment to update the plan.

Plan

What changed

The React Native User Feedback page now includes an iOS sheet troubleshooting subsection. Readers see how to render FeedbackForm above a presented native-stack sheet. sentry-docs must deploy before this page is live. The Vercel project sentry-docs ships this MDX file. develop-docs does not ship this change.

Risk

Bad MDX can drop this page or fail static generation. A broken code fence can throw client errors on this page. The blast radius is this page and other React Native 404s. The React Native SDK does not change.

Intended effect

The new heading must appear in the served HTML. Absent means the page still loads and the heading is missing.

Check Baseline Rule Source
Heading Opening the Feedback Form Over a Presented Sheet on iOS on https://docs.sentry.io/platforms/react-native/user-feedback/ Absent on the live page at 2026-10-27T14:12:00Z Move from absent to present. Absent if docs.page.load still increments and the heading is missing. Production page fetch

Regression watch

This MDX change can 404 the page, empty sibling configuration traffic, or throw in the code block. Page load volume is low and swings day to day.

Check Baseline Rule Source
docs.page.load path /platforms/react-native/user-feedback/ 11 loads in 2026-10-26T14:12:00Z to 2026-10-27T14:12:00Z. 2 loads in 2026-10-25T14:12:00Z to 2026-10-26T14:12:00Z. Band 0.08-0.46 loads per hour. Hold inside 0.08-0.46 per hour. Escalate if this path goes to 0 while the configuration sibling still loads. Sentry sentry/docs metric docs.page.load environment:production path:"/platforms/react-native/user-feedback/"
docs.page.load path /platforms/react-native/user-feedback/configuration/ 1 load in 2026-10-26T14:12:00Z to 2026-10-27T14:12:00Z. 1 load in 2026-10-25T14:12:00Z to 2026-10-26T14:12:00Z. Band 1 load per 24h. Hold at 1-2 loads per 24h. Escalate if this sibling goes to 0 while other platform pages still load. Sentry sentry/docs metric docs.page.load environment:production path:"/platforms/react-native/user-feedback/configuration/"
docs.page.not_found requested_path:platforms/react-native 0 in 2026-10-26T14:12:00Z to 2026-10-27T14:12:00Z. 1 in 2026-10-25T14:12:00Z to 2026-10-27T14:12:00Z. Band 0-1 per 24h. Hold at 0-1 per 24h. Escalate if the count rises above 1. Sentry sentry/docs metric docs.page.not_found environment:production requested_path:"platforms/react-native"
Client errors on this page 1 event in 2026-10-26T14:12:00Z to 2026-10-27T14:12:00Z. 0 extra events in the prior 24h. 1 error per 11 page loads, about 9%. Title Event Event captured as promise rejection. Hold at 0-1 events per 24h and at or below 9% of docs.page.load. Escalate if count rises above 1 on similar load volume, or a new MDX or code-block title appears. Sentry sentry/docs errors environment:production http.url:*platforms/react-native/user-feedback*

If the error rate rises, inspect the new jsx fence and the Alert on this page.

Not observable

Pageload p95 for url.full:*platforms/react-native/user-feedback* was 3752ms in both the 24h and 48h windows. Sample size is 13-17 spans, so that percentile is not a stable band. Soft-navigation spans for this URL returned 0. Transaction name is /:path*?, so path filters on transaction miss this page. Use url.full. docs.page.not_found keeps only the first two path segments, so a 404 on this page is not separable from other React Native 404s. Total docs.page.load for all /platforms/react-native/ paths was not measured, so the 404 check has no RN-wide denominator. docs.snippet_copy truncates page_path to /platforms/react-native and cannot isolate this page.

@antonis
antonis merged commit 6cfc86f into master Oct 27, 2026
28 checks passed
@antonis
antonis deleted the antonis/docs-rn-feedback-over-ios-sheet branch October 27, 2026 14:26
@cursor

cursor Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

Verified in production

  • The live page shows the new iOS sheet heading.

This branch was successfully deployed

1 active deployment
Preview – sentry-docs — 381c3faa Deployed Oct 27, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: Normal Docs review has no urgent deadline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants