Skip to content

docs(replay): clarify fast renderer option docs#6276

Merged
antonis merged 1 commit into
getsentry:mainfrom
leohara:docs/clarify-mobile-replay-fast-renderer-docs
Jun 12, 2026
Merged

docs(replay): clarify fast renderer option docs#6276
antonis merged 1 commit into
getsentry:mainfrom
leohara:docs/clarify-mobile-replay-fast-renderer-docs

Conversation

@leohara

@leohara leohara commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

This updates the enableFastViewRendering JSDoc to reference enableViewRendererV2 instead of the deprecated enableExperimentalViewRenderer.
The runtime behavior is unchanged; this only clarifies the current Mobile Replay iOS option pairing.

💡 Motivation and Context

The previous comment suggested that enableFastViewRendering should be used with enableExperimentalViewRenderer, but that option is deprecated in favor of enableViewRendererV2.
This makes the documentation match the current default option pairing used by Mobile Replay on iOS.

💚 How did you test it?

  • git diff --check
    Not run:
  • yarn workspace @sentry/react-native lint:fmt could not complete because this checkout does not have a node_modules install state.
  • Full build/test suite, because this is a JSDoc-only change.

📝 Checklist

  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

None

@github-actions

github-actions Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Semver Impact of This PR

None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


  • docs(replay): clarify fast renderer option docs by leohara in #6276

🤖 This preview updates automatically when you update the PR.

@lucas-zimerman lucas-zimerman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ご貢献ありがとうございます
LGTM!

@lucas-zimerman lucas-zimerman added skip-changelog ready-to-merge Triggers the full CI test suite labels Jun 12, 2026
@antonis antonis merged commit e333d29 into getsentry:main Jun 12, 2026
57 of 63 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Triggers the full CI test suite skip-changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants