Skip to content

Conversation

@mydea
Copy link
Member

@mydea mydea commented Apr 4, 2025

This updates the "Filtering" page to:

  1. Better separate the breadcrumbs stuff out
  2. Cross-reference the Options & APIs pages instead of repeating all the details everywhere
  3. Clean up some stuff there while we are at it

Also small adjustments to "Releases & Health" page, while at it.

@mydea mydea requested review from a team, AbhiPrasad and lforst April 4, 2025 10:25
@mydea mydea self-assigned this Apr 4, 2025
@vercel
Copy link

vercel bot commented Apr 4, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 11, 2025 9:15am
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
changelog ⬜️ Ignored (Inspect) Visit Preview Apr 11, 2025 9:15am
develop-docs ⬜️ Ignored (Inspect) Visit Preview Apr 11, 2025 9:15am

@codecov
Copy link

codecov bot commented Apr 4, 2025

Bundle Report

Changes will decrease total bundle size by 501 bytes (-0.0%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
sentry-docs-server-cjs 11.13MB -495 bytes (-0.0%) ⬇️
sentry-docs-client-array-push 9.47MB -6 bytes (-0.0%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: sentry-docs-server-cjs

Assets Changed:

Asset Name Size Change Total Size Change (%)
1729.js -3 bytes 1.75MB -0.0%
../instrumentation.js -3 bytes 1.08MB -0.0%
9523.js -3 bytes 1.05MB -0.0%
../app/[[...path]]/page.js.nft.json -162 bytes 384.61kB -0.04%
../app/platform-redirect/page.js.nft.json -162 bytes 384.53kB -0.04%
../app/sitemap.xml/route.js.nft.json -162 bytes 382.5kB -0.04%
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 868.7kB -0.0%
static/chunks/8165-*.js -3 bytes 410.1kB -0.0%
static/zNzv1RIcKNiNX82Mn_yX7/_buildManifest.js (New) 578 bytes 578 bytes 100.0% 🚀
static/zNzv1RIcKNiNX82Mn_yX7/_ssgManifest.js (New) 77 bytes 77 bytes 100.0% 🚀
static/ULfEPTqjE6jwQtrZRZsBj/_buildManifest.js (Deleted) -578 bytes 0 bytes -100.0% 🗑️
static/ULfEPTqjE6jwQtrZRZsBj/_ssgManifest.js (Deleted) -77 bytes 0 bytes -100.0% 🗑️

Comment on lines -67 to -71
<Alert>

Crash reporting and app hang detection are not available for watchOS.

</Alert>
Copy link
Contributor

Choose a reason for hiding this comment

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

Follow up task. Remove this for ruby, powershell, rust, etc. where it doesn't make any sense...

Copy link
Contributor

@coolguyzone coolguyzone left a comment

Choose a reason for hiding this comment

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

LGTM!

mydea and others added 3 commits April 11, 2025 10:19
Also small adjustments to "Releases & Health" page, while at it.
@mydea mydea merged commit 0b0cdb4 into master Apr 11, 2025
11 checks passed
@mydea mydea deleted the fn/link-for-details branch April 11, 2025 09:19
bitsandfoxes pushed a commit that referenced this pull request Apr 24, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Apr 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants