feat: Add Trust is _1 #11596
Merged
feat: Add Trust is _1 #11596
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
lforst
approved these changes
Oct 17, 2024
Lms24
approved these changes
Oct 17, 2024
krystofwoldrich
approved these changes
Oct 17, 2024
sentrivana
approved these changes
Oct 17, 2024
bitsandfoxes
approved these changes
Oct 17, 2024
chargome
approved these changes
Oct 17, 2024
develop-docs/sdk/philosophy.mdx
Outdated
|
|
||
| The role of an SDK is to instrument the user's application, collect context around the event types we want to capture, and send event data with useful context to Sentry. We should avoid extensive business logic within the SDK that transforms this data as it can get complex, hard to maintain and makes permanent changes to the wire format we send events. We should aim to have data collected by the SDK be as raw as possible. We have more flexibility and control on the server to manipulate the data more effectively. | ||
|
|
||
| ## Trust is #1 |
There was a problem hiding this comment.
If it's #1 it should be at the top of the list, IMO.
kahest
approved these changes
Oct 17, 2024
develop-docs/sdk/philosophy.mdx
Outdated
|
|
||
| The role of an SDK is to instrument the user's application, collect context around the event types we want to capture, and send event data with useful context to Sentry. We should avoid extensive business logic within the SDK that transforms this data as it can get complex, hard to maintain and makes permanent changes to the wire format we send events. We should aim to have data collected by the SDK be as raw as possible. We have more flexibility and control on the server to manipulate the data more effectively. | ||
|
|
||
| ## Trust is #1 |
|
|
||
| This document sets some general guidance for how to approach SDK development at Sentry. It should aid both internal and external developers to understand what motivations go into the design of the SDKs and why we're deciding in certain ways. | ||
|
|
||
| ## Trust is #1 |
There was a problem hiding this comment.
Thanks for putting this at #1
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There is some overlap of this in degrade gracefully but we generally feel "stability" by itself deserves a dedicated callout