search

lbryio / lbry-desktop

A browser and wallet for LBRY, the decentralized, user-controlled content marketplace.
https://lbry.tech
MIT License
3.57k stars 414 forks source link

Sort by "Oldest first" in Following only returns max 3 months old claims, instead of the acutally oldest ones #7733

Open keikari opened 1 year ago

keikari commented 1 year ago

claim_search in Following page has release_time: ">3 months ago". Which makes sort by "oldest first" not return oldest claims.
Also when sorting by "newest first", can only find claims up to that time.

oldest-first

https://www.reddit.com/r/lbry/comments/yuewmm/lbry_desktop_client_sorting_by_oldest_first/

dgarrett01 commented 1 year ago

This can be fixed by commenting out or removing these if conditions in ui\component\claimListDiscover\view.jsx...

if ( (options.channel_ids && options.channel_ids.length > 20) || (options.any_tags && options.any_tags.length > 20) ) { options.release_time = >${Math.floor(moment().subtract(3, CS.FRESH_MONTH).startOf('week').unix())}; } else if ( (options.channel_ids && options.channel_ids.length > 10) || (options.any_tags && options.any_tags.length > 10) ) { options.release_time = >${Math.floor(moment().subtract(1, CS.FRESH_YEAR).startOf('week').unix())}; } else { // Hack for at least the New page until https://github.com/lbryio/lbry-sdk/issues/2591 is fixed options.release_time = <${Math.floor(moment().startOf('minute').unix())}; }

jessopb commented 1 year ago

Yeah, I'll release a test to make sure we can greatly reduce or eliminate these.