Skip to content

Conversation

@alwx
Copy link
Contributor

@alwx alwx commented Aug 14, 2025

Time To Initial Display / Time To Full Display works with the new architecture after certain changes done in the past: getsentry/sentry-react-native#5081
It still needs to be re-implemented as a turbo module, and that will be handled later but at least it can be used now.

@vercel
Copy link

vercel bot commented Aug 14, 2025

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

Project Deployment Preview Comments Updated (UTC)
sentry-docs Ready Preview Comment Aug 14, 2025 11:13am
1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
develop-docs ⬜️ Ignored Aug 14, 2025 11:13am

@codecov
Copy link

codecov bot commented Aug 14, 2025

Bundle Report

Changes will decrease total bundle size by 15 bytes (-0.0%) ⬇️. 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.45MB -9 bytes (-0.0%) ⬇️

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 6.55kB 555.3% ⚠️
server/middleware-*.js -5.55kB 1.0kB -84.74%
static/pC5PDFji7qW0MqOOfbEvN/_buildManifest.js (New) 684 bytes 684 bytes 100.0% 🚀
static/pC5PDFji7qW0MqOOfbEvN/_ssgManifest.js (New) 77 bytes 77 bytes 100.0% 🚀
static/Cs4c_58xP67t4t6UHoZMY/_buildManifest.js (Deleted) -684 bytes 0 bytes -100.0% 🗑️
static/Cs4c_58xP67t4t6UHoZMY/_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%

Copy link
Collaborator

@lucas-zimerman lucas-zimerman left a comment

Choose a reason for hiding this comment

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

Thank you for the update!
Let's merge it once a new version with the fix is released

Copy link
Contributor

@antonis antonis left a comment

Choose a reason for hiding this comment

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

LGTM 🚀
Feel free to merge it when getsentry/sentry-react-native#5081 is released.

@alwx alwx self-assigned this Aug 26, 2025
@antonis
Copy link
Contributor

antonis commented Sep 17, 2025

Merging now since getsentry/sentry-react-native#5081 was released with 6.21.0

@antonis antonis merged commit 5620ed2 into master Sep 17, 2025
13 checks passed
@antonis antonis deleted the alwx/docs/ttid-ttfd branch September 17, 2025 08:29
@github-actions github-actions bot locked and limited conversation to collaborators Oct 3, 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.

5 participants