-
Notifications
You must be signed in to change notification settings - Fork 482
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
Code Review Groups: add group management UI to modal #43281
Conversation
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, just a couple nits
body={codeReviewGroupsManager} | ||
isOpen={isDialogOpen} | ||
handleClose={onDialogClose} | ||
fixedWidth={1000} |
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.
nit: constant for 1000?
} | ||
|
||
ManageCodeReviewGroups.propTypes = {buttonContainerStyle: PropTypes.object}; | ||
|
||
const names = [ |
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.
nit: add a todo to remove this fake data at some point
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.
Added a Jira link (and included this in the cleanup task).
Puts rough version of Code Review Groups manager in a modal (hidden behind an experiment flag) where it will ultimately appear. We should continue development of this feature (hooking up to back end, addtional UI work) here rather than in Storybook.
Testing story
Tested manually.