chore: Clarification around size limits and stats #9911
Merged
Conversation
This file contains 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 ↗︎
|
Bundle ReportChanges will increase total bundle size by 168 bytes ⬆️
|
vivianyentran
approved these changes
May 3, 2024
|
I think adding a page on size limits would be great! |
|
@vivianyentran I split out the a size limits page and added it under Data Management - wdyt? |
# Conflicts: # docs/product/stats/index.mdx
|
@vivianyentran pls give this another quick look - I merged in the redesign, recreated the new page under the new concepts section and rewired the links |
Added some headings to make this skimmable
|
Made some edits directly! I'll go ahead and merge this now |
antonpirker
pushed a commit
that referenced
this pull request
May 14, 2024
* adds link to stats, adds size limits drop reason * splits out size limits page under data mgmt * temp remove new file for merge * merges redesign & rewires/recreates content update * Update size-limits.mdx Added some headings to make this skimmable * Update docs/product/stats/index.mdx * Update size-limits.mdx * Update index.mdx fix link --------- Co-authored-by: vivianyentran <20403606+vivianyentran@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
To clarify that events dropped due to size limits are part of the Usage Stats "Dropped" category:
TBD: The size limits docs are currently hard to find (see e.g. getsentry/sentry#70342). I think it'd be reasonable to maybe add a page directly under the "Data Management" tree? Happy to add this to the PR if we agree on something