-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
add updated org stats content for EA #3419
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/sentry/sentry-docs/8NXn5WQbF4AcWy9JaZYNnJPQfpEB |
manuzope
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @imatwawana, think you meant to tag @getsentry/enterprise?
|
|
||
| The page is designed for org admins, but is available to all team members. This is helpful if a team member who isn't an admin is tasked with reviewing these statistics. Also, this allows team members associated with specific projects to use this page to investigate why their events are being dropped. | ||
|
|
||
| [image here] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this should be an image?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I'm just waiting on one, but just wanted to get this PR started in the meantime.
|
Great start. Holding on approval until images are loaded and the team finishes its review. |
Add link and wording edit Co-authored-by: Fiona <[email protected]>
Style edits Co-authored-by: Fiona <[email protected]>
Wording edits Co-authored-by: Fiona <[email protected]>
imatwawana
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Edited to move definitions into intro
Edited to move definitions of accepted, filtered, dropped into intro
leedongwei
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Holding off on an approval because (1) still need screenshots from me, and (2) we may rename "dropped" to "rate-limited".
|
Pushed some changes to the branch to address the EA note and change from "Dropped" to "Rate Limited". Also added a spot for the screenshot you're creating @leedongwei to be dropped in the src/docs/product/accounts/quotas/manage-event-stream-guide.mdx in addition to the new Org Stats page. |
Adding time zone information; changing "time period" to "date range"
Co-authored-by: Danny Lee <[email protected]>
Co-authored-by: Danny Lee <[email protected]>
Adding updated content for Org Stats page for EA