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

[YT] Fix "Could not get like count" #1134

Closed
wants to merge 1 commit into from

Conversation

litetex
Copy link
Member

@litetex litetex commented Nov 30, 2023

Fixes #1133

  1. YT uses a different data structure for likes (ViewModels instead of Renderers)
  2. Fixed NPE in old code due to incorrect usage of string comparison
  3. Unified JSONObject filtering and mapping
New data structure looks like this

data


  • 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.

1. YT uses a different data structure for likes (ViewModels instead of Renderers)
2. Fixed  NPE  in old code due to incorrect usage of string comparison
3. Unified JSONObject filtering and mapping
@litetex litetex marked this pull request as ready for review November 30, 2023 16:27
@AudricV
Copy link
Member

AudricV commented Nov 30, 2023

Hi @litetex, glad to see you again on NewPipe repos 🎉

Unfortunately, your PR is a "duplicate" of one previously closed: #1123. I am closing your PR for the same reason as my comment on this PR. Don't hesitate to let me know what you think of my changes!

As YouTube seems now to roll out this change, I will change my changes in #1127 to try to use the new structure first soon.

Thank you for time and sorry to close your PR :(

@AudricV AudricV closed this Nov 30, 2023
@AudricV AudricV added the duplicate Issue or discussion is a duplicate of an existing issue or discussion label Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate Issue or discussion is a duplicate of an existing issue or discussion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

YT: Could not get like count
2 participants