Skip to content
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

Fix attempt - Sometimes wild-west-like results appear in hometab playlists #3089

Merged
merged 1 commit into from
Apr 22, 2024

Conversation

keikari
Copy link
Contributor

@keikari keikari commented Apr 22, 2024

Fixes

https://secure.helpscout.net/conversation/2570541219/242582?folderId=4588968
User reported about sometimes having some random items showing in their home tab playlists.
There was a generic claim search happening when reloading the channel hometab page(screenshot of the params), and results from that got shown sometimes on some lists before the real items got loaded.

Issue can be seen if reloading this page:
https://odysee.com/@Flaviano_Pasinetti:2
Un-wanted generic claim search is consistent. Seeing the effects on UI isn't consistent.

2024-04-22_11-01

I don't understand why the issue exactly happens, but this seems to fix it. And doesn't seem to break other hometab sections

@keikari keikari changed the title Fix attempt - Sometime wild-west-like results appear in hometab playlists Fix attempt - Sometimes wild-west-like results appear in hometab playlists Apr 22, 2024
@keikari keikari force-pushed the fix-hometab-list-bug branch from 655dccb to cda9380 Compare April 22, 2024 08:35
)
)}
{section.type !== 'channels'
? (collectionUrls || claimSearchResults || collectionClaimIds) && (
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This check was the only change

@tzarebczan
Copy link
Contributor

Thanks for fixing, looks good!

@tzarebczan tzarebczan merged commit 407bfb8 into OdyseeTeam:master Apr 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants