Skip to content
This repository was archived by the owner on Aug 14, 2024. It is now read-only.

Protocols new properties described - #358

Merged
bruno-garcia merged 3 commits into
getsentry:masterfrom
semuserable:feature/protocols-updated
Jun 25, 2021
Merged

Protocols new properties described#358
bruno-garcia merged 3 commits into
getsentry:masterfrom
semuserable:feature/protocols-updated

Conversation

@semuserable

Copy link
Copy Markdown
Contributor

No description provided.

@vercel

vercelBot commented Jun 24, 2021

Copy link
Copy Markdown

@semuserable is attempting to deploy a commit to the Sentry Team on Vercel.

A member of the Team first needs to authorize it.

@semuserable
semuserable marked this pull request as ready for review June 24, 2021 15:37
@semuserable

Copy link
Copy Markdown
ContributorAuthor

Please, review and run authorized deploy for Vercel. Can't add reviewers, btw..

cc @bruno-garcia

@bruno-garcia
bruno-garcia requested review from bitsandfoxes, jan-auer and untitaker and removed request for untitakerJune 25, 2021 19:40
Comment threadsrc/docs/sdk/event-payloads/contexts.mdx Outdated
Comment threadsrc/docs/sdk/event-payloads/contexts.mdx Outdated

@untitakeruntitaker left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Comment threadsrc/docs/sdk/event-payloads/contexts.mdx
@bruno-garcia

Copy link
Copy Markdown
Member

I'd ask you to add them to https://github.com/getsentry/relay/blob/master/relay-general/src/protocol/contexts.rs as well.

Keen to write some Rust? Probably don't even need to compile it locally :D

@bruno-garcia
bruno-garcia merged commit a527407 into getsentry:masterJun 25, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@semuserable@bruno-garcia@untitaker