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
Fix #2874Support ability to select a role as default from the UI #3145
Conversation
[catalog] Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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.
Few changes to take care of in next PR
|
||
export const getUserCounts = () => { | ||
return APIClient.get( | ||
'/search/query?q=*&from=0&size=0&index=user_search_index' |
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.
Use SearchIndex
enum to pass index here
@@ -17,7 +17,7 @@ import { Button } from '../../buttons/Button/Button'; | |||
type Props = { | |||
cancelText: string | ReactNode; | |||
confirmText: string | ReactNode; | |||
bodyText: string; | |||
bodyText: string | ReactNode; |
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.
Should also support Array<ReactNode>
incase of multiple elements in future from some other component.
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.
@darth-coder00 ReactNode
by default support these types.
ReactChild | ReactFragment | ReactPortal | boolean | null | undefined;
So we don't have to do Array<ReactNode>
, ReactNode
will be sufficient.
Closes #2874
Describe your changes :
I worked on issue #2874
Type of change :
Frontend Preview (Screenshots) :
Checklist:
Reviewers
@shahsank3t, @darth-coder00