claim_search: Don't clear previous page results if subsequent pages timeout.
## Issue https://github.com/lbryio/lbry-desktop/issues/4609 ## Change - Don't clear existing results on timeout. - Treat this scenario as "last page reached" by marking `claimSearchByQueryLastPageReached`.
This commit is contained in:
parent
c86810038c
commit
8093d69807
2 changed files with 17 additions and 3 deletions
9
dist/bundle.es.js
vendored
9
dist/bundle.es.js
vendored
|
@ -5155,13 +5155,20 @@ reducers[CLAIM_SEARCH_FAILED] = (state, action) => {
|
||||||
const { query } = action.data;
|
const { query } = action.data;
|
||||||
const claimSearchByQuery = Object.assign({}, state.claimSearchByQuery);
|
const claimSearchByQuery = Object.assign({}, state.claimSearchByQuery);
|
||||||
const fetchingClaimSearchByQuery = Object.assign({}, state.fetchingClaimSearchByQuery);
|
const fetchingClaimSearchByQuery = Object.assign({}, state.fetchingClaimSearchByQuery);
|
||||||
|
const claimSearchByQueryLastPageReached = Object.assign({}, state.claimSearchByQueryLastPageReached);
|
||||||
|
|
||||||
delete fetchingClaimSearchByQuery[query];
|
delete fetchingClaimSearchByQuery[query];
|
||||||
|
|
||||||
|
if (claimSearchByQuery[query] && claimSearchByQuery[query].length !== 0) {
|
||||||
|
claimSearchByQueryLastPageReached[query] = true;
|
||||||
|
} else {
|
||||||
claimSearchByQuery[query] = null;
|
claimSearchByQuery[query] = null;
|
||||||
|
}
|
||||||
|
|
||||||
return Object.assign({}, state, {
|
return Object.assign({}, state, {
|
||||||
fetchingClaimSearchByQuery,
|
fetchingClaimSearchByQuery,
|
||||||
claimSearchByQuery
|
claimSearchByQuery,
|
||||||
|
claimSearchByQueryLastPageReached
|
||||||
});
|
});
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
|
@ -570,13 +570,20 @@ reducers[ACTIONS.CLAIM_SEARCH_FAILED] = (state: State, action: any): State => {
|
||||||
const { query } = action.data;
|
const { query } = action.data;
|
||||||
const claimSearchByQuery = Object.assign({}, state.claimSearchByQuery);
|
const claimSearchByQuery = Object.assign({}, state.claimSearchByQuery);
|
||||||
const fetchingClaimSearchByQuery = Object.assign({}, state.fetchingClaimSearchByQuery);
|
const fetchingClaimSearchByQuery = Object.assign({}, state.fetchingClaimSearchByQuery);
|
||||||
|
const claimSearchByQueryLastPageReached = Object.assign({}, state.claimSearchByQueryLastPageReached);
|
||||||
|
|
||||||
delete fetchingClaimSearchByQuery[query];
|
delete fetchingClaimSearchByQuery[query];
|
||||||
|
|
||||||
|
if (claimSearchByQuery[query] && claimSearchByQuery[query].length !== 0) {
|
||||||
|
claimSearchByQueryLastPageReached[query] = true;
|
||||||
|
} else {
|
||||||
claimSearchByQuery[query] = null;
|
claimSearchByQuery[query] = null;
|
||||||
|
}
|
||||||
|
|
||||||
return Object.assign({}, state, {
|
return Object.assign({}, state, {
|
||||||
fetchingClaimSearchByQuery,
|
fetchingClaimSearchByQuery,
|
||||||
claimSearchByQuery,
|
claimSearchByQuery,
|
||||||
|
claimSearchByQueryLastPageReached,
|
||||||
});
|
});
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue