Fix field filtering, avoid PR call if not needed
This commit is contained in:
parent
efb8388dec
commit
30f8907294
@ -9,7 +9,7 @@ export default function Component({ service }) {
|
|||||||
const { widget } = service;
|
const { widget } = service;
|
||||||
const { userEmail, repositoryId } = widget;
|
const { userEmail, repositoryId } = widget;
|
||||||
const includePR = userEmail !== undefined && repositoryId !== undefined;
|
const includePR = userEmail !== undefined && repositoryId !== undefined;
|
||||||
const { data: prData, error: prError } = useWidgetAPI(widget, "pr");
|
const { data: prData, error: prError } = useWidgetAPI(widget, includePR ? "pr" : null);
|
||||||
const { data: pipelineData, error: pipelineError } = useWidgetAPI(widget, "pipeline");
|
const { data: pipelineData, error: pipelineError } = useWidgetAPI(widget, "pipeline");
|
||||||
|
|
||||||
if (
|
if (
|
||||||
@ -43,26 +43,22 @@ export default function Component({ service }) {
|
|||||||
<Block label="azuredevops.status" value={t(`azuredevops.${pipelineData.value[0].status.toString()}`)} />
|
<Block label="azuredevops.status" value={t(`azuredevops.${pipelineData.value[0].status.toString()}`)} />
|
||||||
)}
|
)}
|
||||||
|
|
||||||
{includePR &&
|
{includePR && <Block label="azuredevops.totalPrs" value={t("common.number", { value: prData.count })} />}
|
||||||
<>
|
{includePR && <Block
|
||||||
<Block label="azuredevops.totalPrs" value={t("common.number", { value: prData.count })} />
|
|
||||||
<Block
|
|
||||||
label="azuredevops.myPrs"
|
label="azuredevops.myPrs"
|
||||||
value={t("common.number", {
|
value={t("common.number", {
|
||||||
value: prData.value?.filter((item) => item.createdBy.uniqueName.toLowerCase() === userEmail.toLowerCase())
|
value: prData.value?.filter((item) => item.createdBy.uniqueName.toLowerCase() === userEmail.toLowerCase())
|
||||||
.length,
|
.length,
|
||||||
})}
|
})}
|
||||||
/>
|
/>}
|
||||||
<Block
|
{includePR && <Block
|
||||||
label="azuredevops.approved"
|
label="azuredevops.approved"
|
||||||
value={t("common.number", {
|
value={t("common.number", {
|
||||||
value: prData.value
|
value: prData.value
|
||||||
?.filter((item) => item.createdBy.uniqueName.toLowerCase() === userEmail.toLowerCase())
|
?.filter((item) => item.createdBy.uniqueName.toLowerCase() === userEmail.toLowerCase())
|
||||||
.filter((item) => item.reviewers.some((reviewer) => reviewer.vote === 10)).length,
|
.filter((item) => item.reviewers.some((reviewer) => reviewer.vote === 10)).length,
|
||||||
})}
|
})}
|
||||||
/>
|
/>}
|
||||||
</>
|
|
||||||
}
|
|
||||||
|
|
||||||
</Container>
|
</Container>
|
||||||
);
|
);
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user