Skip to content

fix: Respect breadcrumbs sentry:false option - #2499

Merged
kamilogorek merged 1 commit into
masterfrom
respect-sentry-breadcrumb-opt
Mar 16, 2020
Merged

fix: Respect breadcrumbs sentry:false option#2499
kamilogorek merged 1 commit into
masterfrom
respect-sentry-breadcrumb-opt

Conversation

@kamilogorek

Copy link
Copy Markdown
Contributor

Fixes #2357

@kamilogorek
kamilogorek requested review from HazAT and rhcarvalho March 16, 2020 12:42
@getsentry-bot

Copy link
Copy Markdown
Contributor
Warnings
⚠️ Please add a changelog entry for your changes.
Messages
📖

@sentry/browser bundle gzip'ed minified size: (ES5: 16.8916 kB) (ES6: 15.9307 kB)

📖 ✅ TSLint passed

Generated by 🚫 dangerJS against 08d44c4

@rhcarvalho rhcarvalho left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Verified that all existing calls to addSentryBreadcrumb are covered.

@kamilogorek
kamilogorek merged commit 39af3b4 into master Mar 16, 2020
@kamilogorek
kamilogorek deleted the respect-sentry-breadcrumb-opt branch March 16, 2020 13:00
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.

Breadcrumb : Sentry events create breadcrumb with parameter sentry = false

3 participants