fix: set initial loading state to true and adjust effect dependencies in RunDetails component
This commit is contained in:
@@ -562,7 +562,7 @@ function RunDetails({
|
|||||||
}) {
|
}) {
|
||||||
const [run, setRun] = useState<RunDetail | null>(null);
|
const [run, setRun] = useState<RunDetail | null>(null);
|
||||||
const [results, setResults] = useState<ResultsResponse | null>(null);
|
const [results, setResults] = useState<ResultsResponse | null>(null);
|
||||||
const [loading, setLoading] = useState(false);
|
const [loading, setLoading] = useState(true);
|
||||||
const [search, setSearch] = useState("");
|
const [search, setSearch] = useState("");
|
||||||
const [verdict, setVerdict] = useState("");
|
const [verdict, setVerdict] = useState("");
|
||||||
const [sellabilityStatus, setSellabilityStatus] = useState("");
|
const [sellabilityStatus, setSellabilityStatus] = useState("");
|
||||||
@@ -599,7 +599,6 @@ function RunDetails({
|
|||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
let cancelled = false;
|
let cancelled = false;
|
||||||
async function loadResults() {
|
async function loadResults() {
|
||||||
setLoading(true);
|
|
||||||
const params = new URLSearchParams({
|
const params = new URLSearchParams({
|
||||||
page: String(page),
|
page: String(page),
|
||||||
pageSize: String(pageSize),
|
pageSize: String(pageSize),
|
||||||
@@ -627,13 +626,17 @@ function RunDetails({
|
|||||||
}, [processType, runId, search, verdict, sellabilityStatus, amazonSellerFilter, minConfidence, maxConfidence, page, pageSize, sort, refreshTick]);
|
}, [processType, runId, search, verdict, sellabilityStatus, amazonSellerFilter, minConfidence, maxConfidence, page, pageSize, sort, refreshTick]);
|
||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
|
if (run && run.status !== "running") {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
const interval = window.setInterval(() => {
|
const interval = window.setInterval(() => {
|
||||||
setRefreshTick((tick) => tick + 1);
|
setRefreshTick((tick) => tick + 1);
|
||||||
}, 4000);
|
}, 4000);
|
||||||
return () => {
|
return () => {
|
||||||
window.clearInterval(interval);
|
window.clearInterval(interval);
|
||||||
};
|
};
|
||||||
}, [processType, runId]);
|
}, [processType, runId, run?.status]);
|
||||||
|
|
||||||
async function reanalyzeItem(item: ResultItem) {
|
async function reanalyzeItem(item: ResultItem) {
|
||||||
const key = String(item.item_id);
|
const key = String(item.item_id);
|
||||||
|
|||||||
Reference in New Issue
Block a user