Skip to content

Conversation

ryan953
Copy link
Member

@ryan953 ryan953 commented Aug 25, 2025

This puts Session Replay in the Nav for the Apple section, and all sub-platforms, just not on iOS anymore.

Also, i moved some FAQ items from the bottom of the index page into troubleshooting.mdx. They seemed hard to find on the index page, and really seem more like

Basically:

  • /platforms/apple/guides/ios/session-replay/ still has all the content as before
  • /platforms/apple/guides/tvos/session-replay/ is a page now! but it's mostly empty
    • (see first screenshots below)
  • /platforms/apple/session-replay/ is a page now! it's a copy of the iOS specific page, but with the faq on the top
    • (see 2nd set of screenshots at end)

/platforms/apple/ Nav Changes:

Before After
SCR-20250825-mjyo SCR-20250825-mkbs

Since most of the Apple platforms are not specifically supported I put the two "Does Session Replay work with *" faq questions at the top of the index page whenever you're not looking specifically at iOS docs.
Only the main Apple and iOS show the sub-pages in the nav bar, the other platform specific sections only show the FAQ stuff and no sub-pages.

Apple Overall iOS Specific The others
SCR-20250825-mird SCR-20250825-misb SCR-20250825-mlcq

Copy link

vercel bot commented Aug 25, 2025

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

Project Deployment Preview Comments Updated (UTC)
develop-docs Ready Ready Preview Comment Aug 27, 2025 10:41pm
sentry-docs Ready Ready Preview Comment Aug 27, 2025 10:41pm

@ryan953 ryan953 requested review from a team August 25, 2025 20:58
@codeowner-assignment codeowner-assignment bot requested a review from a team August 25, 2025 20:58
Copy link

codecov bot commented Aug 25, 2025

Bundle Report

Changes will decrease total bundle size by 15.62kB (-0.07%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
sentry-docs-client-array-push 9.9MB -6 bytes (-0.0%) ⬇️
sentry-docs-server-cjs 12.49MB -15.62kB (-0.12%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: sentry-docs-client-array-push

Assets Changed:

Asset Name Size Change Total Size Change (%)
static/chunks/pages/_app-*.js -3 bytes 882.71kB -0.0%
static/chunks/8321-*.js -3 bytes 425.87kB -0.0%
server/middleware-*.js -5.55kB 1.0kB -84.74%
server/middleware-*.js 5.55kB 6.55kB 555.3% ⚠️
static/X5sAX5qmY26JyG9c6JFd8/_buildManifest.js (New) 684 bytes 684 bytes 100.0% 🚀
static/X5sAX5qmY26JyG9c6JFd8/_ssgManifest.js (New) 77 bytes 77 bytes 100.0% 🚀
static/dtJEYkklbJcEt1FHttI3G/_buildManifest.js (Deleted) -684 bytes 0 bytes -100.0% 🗑️
static/dtJEYkklbJcEt1FHttI3G/_ssgManifest.js (Deleted) -77 bytes 0 bytes -100.0% 🗑️
view changes for bundle: sentry-docs-server-cjs

Assets Changed:

Asset Name Size Change Total Size Change (%)
1729.js -3 bytes 1.78MB -0.0%
../instrumentation.js -3 bytes 1.1MB -0.0%
9523.js -3 bytes 1.08MB -0.0%
../app/[[...path]]/page.js.nft.json -5.2kB 765.92kB -0.67%
../app/platform-redirect/page.js.nft.json -5.2kB 765.83kB -0.67%
../app/sitemap.xml/route.js.nft.json -5.2kB 763.3kB -0.68%

@jas-kas
Copy link
Member

jas-kas commented Aug 26, 2025

Sharing my feedback from Slack

This is what I'd change:

  • Add an Apple > Session Replay page (reasoning: since iOS is under Apple, this is fair game)
    -- I would not lead with a FAQ on the homepage here, but instead simply add a banner or bolded phrase somewhere that this is for the iOS SDK only. Alternatively, link out to iOS docs for more information.

  • I would not add a Session Replay page for macOS, tvOS, watchOS, visionOS simply because they aren't supported platforms. Why have a mostly empty page saying that? I think we should just exclude a Session Replay left-nav item altogether for these platforms imo

I think we can continue to massaging the Troubleshooting and mini-FAQ content in a follow up

added an Alert on top of all pages inside Apple > Session Replay, no content visible
Apple > iOS > Session Replay pages are unchanged
Apple > macOS > Session Replay pages are mostly empty, only show the alert
@ryan953 ryan953 force-pushed the ryan953/replay-apple-all-platforms branch from c7a2f3e to dc09178 Compare August 26, 2025 22:12
cursor[bot]

This comment was marked as outdated.

@ryan953 ryan953 merged commit 7ffc130 into master Aug 27, 2025
13 checks passed
@ryan953 ryan953 deleted the ryan953/replay-apple-all-platforms branch August 27, 2025 23:08
codyde added a commit that referenced this pull request Aug 28, 2025
…om/getsentry/sentry-docs into feat/python-docs-improvement-clean

* 'feat/python-docs-improvement-clean' of https://github.com/getsentry/sentry-docs:
  docs(replay): Bring replay nav item to all apple platforms (#14710)
  User Feedback Basics Page (#14670)
  Update AM3 and legacy billing for logs GA, DO NOT MERGE BEFORE LOGS GA (#14589)
  Add LLMSteering component for hidden documentation instructions (#14744)
  New data retention policy page (#14661)
  Adding error prediction to Prevent AI docs (#14641)
codyde added a commit that referenced this pull request Aug 28, 2025
…om/getsentry/sentry-docs into feat/nextjs-centralized-flow-clean

* 'feat/nextjs-centralized-flow-clean' of https://github.com/getsentry/sentry-docs:
  Fix reference to span in processOrderBatch function (#14741)
  chore(java): Add docs on how to use tracePropagationTargets (#14752)
  docs(replay): Bring replay nav item to all apple platforms (#14710)
  User Feedback Basics Page (#14670)
  Update AM3 and legacy billing for logs GA, DO NOT MERGE BEFORE LOGS GA (#14589)
  Add LLMSteering component for hidden documentation instructions (#14744)
  New data retention policy page (#14661)
  Adding error prediction to Prevent AI docs (#14641)
codyde added a commit that referenced this pull request Aug 29, 2025
* 'master' of https://github.com/getsentry/sentry-docs: (72 commits)
  docs(replay): Bring replay nav item to all apple platforms (#14710)
  User Feedback Basics Page (#14670)
  Update AM3 and legacy billing for logs GA, DO NOT MERGE BEFORE LOGS GA (#14589)
  Add LLMSteering component for hidden documentation instructions (#14744)
  New data retention policy page (#14661)
  Adding error prediction to Prevent AI docs (#14641)
  feat(logs): Remove beta note from logs docs (#14686)
  First pass at logs quota doc (#14671)
  docs(aws): Remove manual setup steps for ESM (#14738)
  Android Logs docs without Errors / Breadcrumbs distractions (#14733)
  chore(Unity): Dropped 2019 relevant section from Known Limitations (#14735)
  feat: Added Unity `UserFeedback` Prefab (#14667)
  chore(Unity): Added Mono & Line Numbers to Known Limitations (#14736)
  docs(js): Fix Next.js client tracing config (#14734)
  chore: Updated known limitation in Unity with Cysharp (#14666)
  feat(Unity ): Added `APK+OBB` initialization to troubleshooting (#14668)
  Codecov copy & screenshot edits (#14659)
  fix(cloudflare): Remove unused instrument.mjs from hono on cloudflare (#14723)
  update advanced scrubbing doc - adding logs section (#14696)
  fixing some errors from nextjs-source-map-updates branch (#14467)
  ...
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.

4 participants