Skip to content

fix(attachments): Add missing view_hierarchy attachment type#11197

Merged
krystofwoldrich merged 2 commits intodevelopfrom
kw-add-view-hierarchy-attachment
Mar 20, 2024
Merged

fix(attachments): Add missing view_hierarchy attachment type#11197
krystofwoldrich merged 2 commits intodevelopfrom
kw-add-view-hierarchy-attachment

Conversation

@krystofwoldrich
Copy link
Copy Markdown
Contributor

Adds missing view hierarchy attachment type used in React Native.

@krystofwoldrich krystofwoldrich requested review from a team, AbhiPrasad, Lms24 and timfish and removed request for a team March 19, 2024 09:45
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 19, 2024

size-limit report 📦

Path Size
@sentry/browser (incl. Tracing, Replay, Feedback) 80.71 KB (added)
@sentry/browser (incl. Tracing, Replay) 72.06 KB (added)
@sentry/browser (incl. Tracing, Replay with Canvas) 75.86 KB (added)
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 65.62 KB (added)
@sentry/browser (incl. Tracing) 36.66 KB (added)
@sentry/browser (incl. browserTracingIntegration) 36.66 KB (added)
@sentry/browser (incl. feedbackIntegration) 31.33 KB (added)
@sentry/browser (incl. feedbackModalIntegration) 31.45 KB (added)
@sentry/browser (incl. feedbackScreenshotIntegration) 31.46 KB (added)
@sentry/browser (incl. sendFeedback) 27.41 KB (added)
@sentry/browser 22.58 KB (added)
CDN Bundle (incl. Tracing, Replay, Feedback) 75.05 KB (added)
CDN Bundle (incl. Tracing, Replay) 69.9 KB (added)
CDN Bundle (incl. Tracing) 36.25 KB (added)
CDN Bundle 23.95 KB (added)
CDN Bundle (incl. Tracing, Replay) - uncompressed 219.54 KB (added)
CDN Bundle (incl. Tracing) - uncompressed 109.49 KB (added)
CDN Bundle - uncompressed 70.96 KB (added)
@sentry/react (incl. Tracing, Replay) 72.05 KB (added)
@sentry/react 22.61 KB (added)

Copy link
Copy Markdown
Contributor

@AbhiPrasad AbhiPrasad left a comment

Choose a reason for hiding this comment

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

Needs to be part of v7 as well?

@AbhiPrasad
Copy link
Copy Markdown
Contributor

Let's also make sure we update https://develop.sentry.dev/sdk/envelopes/#attachment

@krystofwoldrich
Copy link
Copy Markdown
Contributor Author

Needs to be part of v7 as well?

Yes, is there v7 branch or how do we backport items?

@krystofwoldrich
Copy link
Copy Markdown
Contributor Author

@Lms24
Copy link
Copy Markdown
Member

Lms24 commented Mar 20, 2024

Yes, is there v7 branch or how do we backport items?

Yes, I recommend cherry-picking the commit and opening a second PR against v7 for it :)

@krystofwoldrich krystofwoldrich merged commit 3ee65ab into develop Mar 20, 2024
@krystofwoldrich krystofwoldrich deleted the kw-add-view-hierarchy-attachment branch March 20, 2024 20:32
cadesalaberry pushed a commit to cadesalaberry/sentry-javascript that referenced this pull request Apr 19, 2024
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.

4 participants