-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
feat(growth): Alert Bar on issue page for sample events #28864
Conversation
static/app/components/alertBar.tsx
Outdated
font-size: 14px; | ||
`; | ||
|
||
export default AlertBar; |
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.
This sounds pretty generic. Should this have a story in storybook?
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.
@Neo-Zhixing Agreed with Evan, maybe we can make a more specific name?
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.
Any recommendations? (naming is hard)
ActionableAlertBar
? PageAlertBar
?
Btw this is now added to the storybook.
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.
PageAlertBar works for me
Co-authored-by: Alberto Leal <mail4alberto@gmail.com>
@Neo-Zhixing can you include a screenshot? |
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.
@Neo-Zhixing looks good to me, I would let support know about this change in case there are any questions related to it
When the user is looking at a sample error, show a banner that takes the user to the "Getting Started" page in the app.