[HELP NEEDED] Major planned/missing features

[InfinityLoop1308]

List:

• Merge upstream#8837: We have already applied its previous version(i.e. the third UI) so that it is just a UI change and there is not any missing function. The problem is, he refactored his previous code, including a rebase and force push and now his branch is far away from us. Need to solve tons of conflicts. Check branch merging-search-client and meging-search-extractor to continue work on this. ⚠️Warning: head of that branch can not build without solving errors.
• Merge upstream#9410: This pr provides full support of comment replies, while the current implement can only fetch 1 page. Conflicts has been solved, but the problem is, his pr is buggy. Replies after the 1st page will not load and loading indicator is always shown. Further investigation shows that is because onScroll is not called at all, which might because he is using a RecycleView inside another RecycleView. If you have knowledge about android UI, please help me fix this. Check branch merging-comment-replies.
• [Bug]: can not play encrypted HLS video streams #10 Check the issue for more details
• TODO: login #12 It seems we can not gain any benifit using a non-premium account, but most premium features are not supported yet.
• [Bug][Need refactor] can not download BiliBili videos #20
• [Feature request] Better appearance of superchat and superchat history #50 This may need to be implemented with a new UI for Superchats.

[ShareASmile]

@InfinityLoop1308 somewhat Buggy PR #9410 for comments replies has been superceded now by latest clean simpler implementation of Comments Replies by Stypox in upstream NewPipe PR #10018
Please check:
https://github.com/TeamNewPipe/NewPipe/pulls

[InfinityLoop1308]

@InfinityLoop1308 somewhat Buggy PR #9410 for comments replies has been superceded now by latest clean simpler implementation of Comments Replies by Stypox in upstream NewPipe PR #10018 Please check: https://github.com/TeamNewPipe/NewPipe/pulls

Thanks for your information. In this repo the #9410 has been fixed and works perfectly now so there is no need to merge the upstream. (Also I think our implementation is actually better)