LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

Re-poll doFetchFeaturedUris every so often #1090

Closed
daovist wants to merge 1 commit from repoll-featured-uris into master
pull from: repoll-featured-uris
merge into: LBRYCommunity:master
Conversation 2 Commits 1 Files changed 1 +7
1 changed files with 7 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 032f112ae1 - Show all commits

7
src/renderer/redux/actions/content.js
View file

@ -70,6 +70,13 @@ export function doResolveUri(uri) {
} }
export function doFetchFeaturedUris() { export function doFetchFeaturedUris() {
return dispatch => {
dispatch(doFetchFeaturedUrisNow());
setInterval(() => dispatch(doFetchFeaturedUrisNow()), 40000);
}
}
export function doFetchFeaturedUrisNow() {
liamcardenas commented 2018-03-27 04:18:59 +02:00 (Migrated from github.com)
Review
Copy link

Setting an interval like this is absolutely correct, but it would be better if you found the place that originally triggers doFetchFeaturedUris and set the interval there instead of creating two actions.

Setting an interval like this is absolutely correct, but it would be better if you found the place that originally triggers `doFetchFeaturedUris` and set the interval there instead of creating two actions.
return dispatch => { return dispatch => {
dispatch({ dispatch({
type: ACTIONS.FETCH_FEATURED_CONTENT_STARTED, type: ACTIONS.FETCH_FEATURED_CONTENT_STARTED,