Skip to content

ref(nextjs): Remove metadata builder class - #4263

Merged
AbhiPrasad merged 3 commits into
masterfrom
abhi-metdata-builder-next
Dec 14, 2021
Merged

ref(nextjs): Remove metadata builder class#4263
AbhiPrasad merged 3 commits into
masterfrom
abhi-metdata-builder-next

Conversation

@AbhiPrasad

Copy link
Copy Markdown
Contributor

Refactor the class into functions to save on bundle size. Extracted from
changes in #4196.

@github-actions

github-actionsBot commented Dec 10, 2021

Copy link
Copy Markdown
Contributor

size-limit report

PathBase Size (010d37f)Current SizeChange
@sentry/browser - CDN Bundle (gzipped)22.44 KB22.44 KB+0.02% 🔺
@sentry/browser - Webpack23.29 KB23.29 KB0%
@sentry/browser - Webpack - gzip = false82.62 KB82.62 KB0%
@sentry/react - Webpack23.32 KB23.32 KB0%
@sentry/nextjs Client - Webpack48.03 KB47.96 KB-0.15% 🔽
@sentry/browser + @sentry/tracing - CDN Bundle (gzipped)29.96 KB29.96 KB0%

@iker-barriocanaliker-barriocanal 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.

There should be tests, why are we deleting them?

Comment threadpackages/nextjs/src/utils/metadata.ts Outdated
Refactor the class into functions to save on bundle size. Extracted from
changes in #4196.
@AbhiPrasad
AbhiPrasad enabled auto-merge (squash) December 14, 2021 17:31
@AbhiPrasad
AbhiPrasad merged commit e6a300b into masterDec 14, 2021
@AbhiPrasad
AbhiPrasad deleted the abhi-metdata-builder-next branch December 14, 2021 17:46
onurtemizkan pushed a commit that referenced this pull request Dec 19, 2021
* ref(nextjs): Remove metadata builder class
Refactor the class into functions to save on bundle size. Extracted from
changes in #4196.
* add tests for nextjs metadata
Sign up for freeto 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.

2 participants

@AbhiPrasad@iker-barriocanal