-
-
Notifications
You must be signed in to change notification settings - Fork 47
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
feat: Handle deletion and unavailable videos on Youtube #1070
Comments
Should we detect that automatically or let user signal that? |
Goal would be automated detection (for example at the same time that we currently refresh a video metadata). If that does not work, then as a last resort letting user signal it, but then we still need trusted moderators to verifies the signals. |
|
We are going to work with @OoIwazaruoO on the What is the expected behaviour after a click on expand? We display the Tournesol and YouTube metadata as usual? I think we may want to not display the video title, because maybe the video was deleted because of its polemic content and the title itself may be polemic too. How about displaying everything, except the title? The title could be "This video is no accessible anymore". |
If the title is not displayed, and the thumbnail is not available either, is it worth displaying any other metadata at all? The user would not be able to recognize the video, even if they compared it in the past. In this case, I would just keep a minimal collapsed video card without the possibility to expand it. |
I'm wondering if we have reasons to even display a collapsed entity card in the recommendations. Is it really useful to know there were an entity that was at this position in the ranking, but now it is not available on YT? We could simplify the recommendations by simply not recommending non available content. The admin can still see the video in the back end, but not the front end users. We could still display this folded entity card elsewhere, like in the rate-later list, to inform them that what they were about to compare is not comparable anymore. What do you think @amatissart @lfaucon @aidanjungo @OoIwazaruoO ? proposition
|
I took some screenshots to better visualize the recommendations. As a user knowing there was a video here, with an unknown name, that is not available anymore doesn't seem relevant or useful. It can be confusing also because it looks like a title indicating the video above or below is not available. Another capture with this time the poll metadata. It doesn't look very useful too, except if we want the users to be able to go to the analysis page. |
Removing the unavailable video from the recommendations page completely at render time could bring some confusion due to how the pagination works: a page with strictly less than 20 is supposed to be the last page. So I think it would still be relevant to display a compact generic message "This video is not available" to keep the pagination consistent with what the user expects. |
Here is the current state of this feature:
The next steps could be:
capture (a) |
Based on impact, priotities should be (1) hide unavailable videos in the extension recommendations (2) reload in tutorial suggestions (3) reload in home page comparison (...last) comparison page |
I move the issue from In progress to Selected because no one is currently working on it. |
@GresilleSiffle @lfaucon I'll take care of (1) if nobody is working on this already |
No one is working on it as far as I know : ) |
It's not very clear yet all the parts of Tournesol that are impacted and should be impacted in the case of a video deleted by the content creator or removed from Youtube for any reason. This nevertheless gives bad user experience, especially if one such video comes up during the tutorial.
The text was updated successfully, but these errors were encountered: