Skip to content

Conversation

@ryan953
Copy link
Member

@ryan953 ryan953 commented Mar 25, 2025

@vercel
Copy link

vercel bot commented Mar 25, 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 Mar 25, 2025 10:41pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
changelog ⬜️ Ignored (Inspect) Visit Preview Mar 25, 2025 10:41pm
develop-docs ⬜️ Ignored (Inspect) Visit Preview Mar 25, 2025 10:41pm

@codecov
Copy link

codecov bot commented Mar 25, 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-server-cjs 10.68MB -9 bytes (-0.0%) ⬇️
sentry-docs-client-array-push 9.98MB -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 (%)
7517.js -3 bytes 1.66MB -0.0%
../instrumentation.js -3 bytes 910.93kB -0.0%
7688.js -3 bytes 886.9kB -0.0%
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 893.8kB -0.0%
static/chunks/8429-*.js -3 bytes 371.26kB -0.0%
server/middleware-*.js -5.54kB 1.01kB -84.55%
server/middleware-*.js 5.54kB 6.55kB 547.23% ⚠️
static/58umfYSRqRFL6FGHCGQFf/_buildManifest.js (New) 581 bytes 581 bytes 100.0% 🚀
static/58umfYSRqRFL6FGHCGQFf/_ssgManifest.js (New) 77 bytes 77 bytes 100.0% 🚀
static/FCaTCMTTvLp8tOd9ZkdTt/_buildManifest.js (Deleted) -581 bytes 0 bytes -100.0% 🗑️
static/FCaTCMTTvLp8tOd9ZkdTt/_ssgManifest.js (Deleted) -77 bytes 0 bytes -100.0% 🗑️

| `domId (optional)` | `string \| undefined` | The `id` given to the \<div\> that is created to contain the Toolbar html. | `'sentry-toolbar'` |
| `debug (optional)` | `string \| undefined` | A comma separated string of debug targets to enable. Example: `'logging,state'`. If the list contains 'all' or 'true' then all targets will be enabled. Valid targets: `'logging' 'login-success' 'settings' 'state'` | `undefined` |
| `mountPoint (optional)` | `HTMLElement \| () => HTMLElement \| undefined` | Where to mount the Toolbar in the DOM. | `document.body` |
| `transactionToSearchTerm (optional)` | `(transactionName: string) => string` | Hook to transform transactionName into a search term | See [transactionToSearchTerm.ts](https://github.com/getsentry/sentry-toolbar/blob/main/src/lib/utils/transactionToSearchTerm.ts) |
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
| `transactionToSearchTerm (optional)` | `(transactionName: string) => string` | Hook to transform transactionName into a search term | See [transactionToSearchTerm.ts](https://github.com/getsentry/sentry-toolbar/blob/main/src/lib/utils/transactionToSearchTerm.ts) |
| `transactionToSearchTerm (optional)` | `(transactionName: string) => string` | Hook to transform `transactionName` into a search term | See [transactionToSearchTerm.ts](https://github.com/getsentry/sentry-toolbar/blob/main/src/lib/utils/transactionToSearchTerm.ts) |

@ryan953 ryan953 merged commit dcf0d55 into master Mar 31, 2025
11 checks passed
@ryan953 ryan953 deleted the ryan953/231-document-new-config-field branch March 31, 2025 20:45
@github-actions github-actions bot locked and limited conversation to collaborators Apr 16, 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