-
Notifications
You must be signed in to change notification settings - Fork 3k
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
one bug of computing Jobs issues nums #4787
Comments
@lyq12580 why did you close the issue? Your solution works, but the bug is still in the repo. Could you open the issue, please? I could make a pull request, if you wish. |
@alex4men , please do! We love contribution. |
nobody can reply me, so i close it. Now it is already opened |
please help make a pull request ,Thanks |
My actions before raising this issue
find a bug in "cvat\cvat-ui\src\components\task-page\job-list.tsx"
Expected Behaviour
Current Behaviour
useEffect(() => {
setError(null);
jobInstance
.issues(jobInstance.id)
.then((issues: any[]) => {
setSummary({
issues_unsolved: issues.filter((issue) => !issue.resolved_date).length,
issues_resolved: issues.filter((issue) => issue.resolved_date).length,
});
})
.catch((_error: any) => {
// eslint-disable-next-line
console.log(_error);
setError(_error);
});
}, []);
Possible Solution
useEffect(() => {
setError(null);
jobInstance
.issues(jobInstance.id)
.then((issues: any[]) => {
setSummary({
issues_unsolved: issues.filter((issue) => !issue.resolved).length,
issues_resolved: issues.filter((issue) => issue.resolved).length,
});
})
.catch((_error: any) => {
// eslint-disable-next-line
console.log(_error);
setError(_error);
});
}, []);
Steps to Reproduce (for bugs)
Context
Your Environment
git log -1
):docker version
(e.g. Docker 17.0.05):Logs from `cvat` container
Next steps
You may join our Gitter channel for community support.
The text was updated successfully, but these errors were encountered: