-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Stream details outputs #19685
Stream details outputs #19685
Conversation
d827160
to
10d908d
Compare
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.
Code looks good to me, found one nitpick.
I will test next.
}; | ||
|
||
const ActionButtonsWrap = styled.span` | ||
margin-right: 6px; |
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.
Can we use theme spacings here?
53ab358
to
23f19a1
Compare
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.
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.
340d735
to
b6dbf68
Compare
74b764c
to
4f338ca
Compare
- with edit and remove outputs
4f338ca
to
7c4c6dd
Compare
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
tested adding, editing, assigning & deleting outputs
Add the outputs section on the stream details page destination segment.
/nocl
Types of changes
Checklist: