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

Load more featured pages and fix featured cover arts not loading #589

Merged
merged 3 commits into from
Mar 31, 2021

Conversation

fynngodau
Copy link
Member

@fynngodau fynngodau commented Mar 28, 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.

Closes #584
→ NewPipe now allows scrolling the Bandcamp Featured screen by more than 10 items by loading additional pages.

Fixes #587
→ As described there, use item_art_id if art_id is not available. This fixes broken cover arts on the featured screen.

I verified against the Bandcamp Inc. app that the featured screen displays the same items (besides tracks, #588).

@fynngodau fynngodau added the bandcamp service, https://bandcamp.com label Mar 28, 2021
@AudricV AudricV added enhancement New feature or request bug Issue is related to a bug labels Mar 28, 2021
Copy link
Contributor

@TobiGr TobiGr left a comment

Choose a reason for hiding this comment

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

Looks good. Just one small thing.

@fynngodau fynngodau force-pushed the bandcamp-more-recent-pages branch from b161ecc to 705f6c6 Compare March 31, 2021 09:02
@TobiGr TobiGr merged commit 7ea2cd7 into TeamNewPipe:dev Mar 31, 2021
@fynngodau fynngodau deleted the bandcamp-more-recent-pages branch October 15, 2021 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bandcamp service, https://bandcamp.com bug Issue is related to a bug enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Some featured albums not showing correct thumbnail Multiple pages in featured screen
3 participants