Skip to content

feat(server): Update multipart infer for view-hierarchy.json#4624

Merged
JoshuaMoelans merged 6 commits intomasterfrom
joshua/feat/infer-view-hierarchy
Apr 1, 2025
Merged

feat(server): Update multipart infer for view-hierarchy.json#4624
JoshuaMoelans merged 6 commits intomasterfrom
joshua/feat/infer-view-hierarchy

Conversation

@JoshuaMoelans
Copy link
Copy Markdown
Member

@JoshuaMoelans JoshuaMoelans commented Mar 28, 2025

Related to getsentry/sentry-native#1191

We want to infer the AttachmentType of any incoming attachment that is called view-hierarchy.json. This follows the file naming convention specified in RFC#33. We can still match on the field_name because it'll be the same as the file_name.

it is the same as the file_name, so we keep it consistent
@JoshuaMoelans JoshuaMoelans marked this pull request as ready for review April 1, 2025 09:42
@JoshuaMoelans JoshuaMoelans requested a review from a team as a code owner April 1, 2025 09:42
@tobias-wilfert
Copy link
Copy Markdown
Member

Please add a changelog entry, but other than that this looks good.

@JoshuaMoelans JoshuaMoelans changed the title update infer for view-hierarchy.json feat(server): Update multipart infer for view-hierarchy.json Apr 1, 2025
@JoshuaMoelans JoshuaMoelans changed the title feat(server): Update multipart infer for view-hierarchy.json feat(server): Update multipart infer for view-hierarchy.json Apr 1, 2025
@JoshuaMoelans JoshuaMoelans merged commit 771a8d0 into master Apr 1, 2025
25 checks passed
@JoshuaMoelans JoshuaMoelans deleted the joshua/feat/infer-view-hierarchy branch April 1, 2025 11:18
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.

3 participants