Skip to content

fix(browser): Change UserAgent export to HttpContext #5033

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
May 4, 2022

Conversation

Lms24
Copy link
Member

@Lms24 Lms24 commented May 4, 2022

After merging #5027, I discovered that I didn't rebase the PR after the integrations exports change in #5028 exported UserAgent. Should have double checked before merging. Anyway, this PR renames the export of the UserAgent integration to HttpContext which should fix building.

@Lms24 Lms24 requested review from AbhiPrasad and lforst May 4, 2022 11:41
@Lms24 Lms24 changed the title fix(browser): change UserAgent export to HttpContext fix(browser): Change UserAgent export to HttpContext May 4, 2022
@github-actions
Copy link
Contributor

github-actions bot commented May 4, 2022

size-limit report 📦

Path Size
@sentry/browser - ES5 CDN Bundle (gzipped + minified) 18.74 KB (-6.96% 🔽)
@sentry/browser - ES5 CDN Bundle (minified) 58.56 KB (-9.37% 🔽)
@sentry/browser - ES6 CDN Bundle (gzipped + minified) 17.55 KB (-6.95% 🔽)
@sentry/browser - ES6 CDN Bundle (minified) 52.74 KB (-9.03% 🔽)
@sentry/browser - Webpack (gzipped + minified) 19.21 KB (-17.35% 🔽)
@sentry/browser - Webpack (minified) 62.08 KB (-24.03% 🔽)
@sentry/react - Webpack (gzipped + minified) 19.23 KB (-17.4% 🔽)
@sentry/nextjs Client - Webpack (gzipped + minified) 42.9 KB (-10.72% 🔽)
@sentry/browser + @sentry/tracing - ES5 CDN Bundle (gzipped + minified) 24.51 KB (-5.99% 🔽)
@sentry/browser + @sentry/tracing - ES6 CDN Bundle (gzipped + minified) 23.09 KB (-5.67% 🔽)

@lforst lforst merged commit 794f65b into 7.x May 4, 2022
@lforst lforst deleted the lms-fix-useragent-export branch May 4, 2022 12:06
@AbhiPrasad AbhiPrasad added this to the 7.0.0 milestone May 12, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants