fix(graph): fix empty target group (#26704)
<!-- Please make sure you have read the submission guidelines before posting an PR --> <!-- https://github.com/nrwl/nx/blob/master/CONTRIBUTING.md#-submitting-a-pr --> <!-- Please make sure that your commit message follows our format --> <!-- Example: `fix(nx): must begin with lowercase` --> <!-- If this is a particularly complex change or feature addition, you can request a dedicated Nx release for this pull request branch. Mention someone from the Nx team or the `@nrwl/nx-pipelines-reviewers` and they will confirm if the PR warrants its own release for testing purposes, and generate it for you if appropriate. --> ## Current Behavior <!-- This is the behavior we have today -->  when the others group are empty, it shows the container. move the check to graph/ui-project-details/src/lib/target-configuration-details-group-container/target-configuration-details-group-container.tsx. ## Expected Behavior <!-- This is the behavior we should expect with the changes in this PR --> ## Related Issue(s) <!-- Please link the issue being fixed so it gets closed when this is merged. --> Fixes #
This commit is contained in:
parent
4197a91845
commit
5e022951da
@ -17,6 +17,9 @@ export function TargetConfigurationGroupContainer({
|
||||
nxConnectCallback,
|
||||
children,
|
||||
}: TargetConfigurationGroupContainerProps) {
|
||||
if (targetsNumber === 0) {
|
||||
return null;
|
||||
}
|
||||
return (
|
||||
<div className="mb-4 w-full">
|
||||
<TargetConfigurationGroupHeader
|
||||
|
||||
@ -47,9 +47,6 @@ export function TargetConfigurationGroupList({
|
||||
<>
|
||||
{Object.entries(targetsGroup.groups).map(
|
||||
([targetGroupName, targets]) => {
|
||||
if (targets.length === 0) {
|
||||
return null;
|
||||
}
|
||||
return (
|
||||
<TargetConfigurationGroupContainer
|
||||
targetGroupName={targetGroupName}
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user