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

[YouTube] Fix missing comments under videos #572

Merged
merged 3 commits into from
Mar 5, 2021
Merged

[YouTube] Fix missing comments under videos #572

merged 3 commits into from
Mar 5, 2021

Conversation

golfinq
Copy link
Contributor

@golfinq golfinq commented Mar 5, 2021

  • I carefully read the contribution guidelines and agree to them.
  • I have tested the API against NewPipe.
  • I agree to create a pull request for NewPipe as soon as possible to make it compatible with the changed API.

Fixes TeamNewPipe/NewPipe#5585, TeamNewPipe/NewPipe#5757, TeamNewPipe/NewPipe#5759.

It seems youtube is in the process of slightly changing their API, and the comment extractor could no longer find the ctoken. Based on reports that the old way sometimes worked, I left it in as an option. I have tested it on my phone and the app works correctly with this change.

@golfinq
Copy link
Contributor Author

golfinq commented Mar 5, 2021

The failed tests seem to concern parts of the code which this PR does not alter; I am not sure why it is failing.

@TobiGr
Copy link
Contributor

TobiGr commented Mar 5, 2021

Thank you!
The other test are failing, because YouTube made further changes to the API, e.g. to the continuations. SoundCloud is failing as well, but that will be fixed by nother PR, too.

@TobiGr TobiGr merged commit b608587 into TeamNewPipe:dev Mar 5, 2021
@TobiGr TobiGr added bug Issue is related to a bug youtube service, https://www.youtube.com/ labels Mar 5, 2021
@AudricV AudricV changed the title Fixes missing comments under videos [YouTube] Fix missing comments under videos Mar 5, 2021
@golfinq golfinq deleted the fixed-comments branch March 11, 2021 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue is related to a bug youtube service, https://www.youtube.com/
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants