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
ui: Add create new team button (PROJQUAY-5685) #2309
Conversation
Codecov ReportAll modified lines are covered by tests ✅ see 2 files with indirect coverage changes 📢 Thoughts on this report? Let us know!. |
b67c4b3
to
0511c61
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.
Tested locally, works good. The only thing is when creating the team I see the new row but the values in the row are 0 repositories and 0 members until i refresh the page. Is it possible to fix this and the other comments within this PR?
</Conditional> | ||
<Conditional if={props.isTeamWizardOpen}> | ||
{props.createTeamWizard} | ||
</Conditional> |
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.
Couldn't these modals be rendered in the parent component? That way you don't need the state props for whether they're open, you only pass the handleModalToggle
. You can then also remove the conditional renders from all the *ViewList.tsx
components.
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.
moved it to parent component. Code looks much cleaner now with lesser props being passed down. Thanks!
...src/routes/OrganizationsList/Organization/Tabs/TeamsAndMembership/TeamsAndMembershipList.tsx
Outdated
Show resolved
Hide resolved
0511c61
to
be64e9c
Compare
Signed-off-by: harishsurf <hgovinda@redhat.com>
be64e9c
to
71172db
Compare
71172db
to
af53238
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
Adds create new team button under teams and membership tab: