Skip to content

Conversation

danielebogo
Copy link
Contributor

@danielebogo danielebogo commented Aug 19, 2025

📘 Part of: Playlists

Fixes PCIOS-94

This PR increases the limit of the displayed episodes for Playlists

To test

  • Make sure you have the rebranding ff on
  • Make sure you have a big database with many podcasts to reach more than 1000 episodes
  • Create a filter with all podcasts
  • The limit should be 1000
  • Do the test with the FF off and confirm the limit status at 500
  • To check the limit add a breakpoint in func queryFor(filter: EpisodeFilter, episodeUuidToAdd: String?, limit: Int) -> String

Checklist

  • I have considered if this change warrants user-facing release notes and have added them to CHANGELOG.md if necessary.
  • I have considered adding unit tests for my changes.
  • I have updated (or requested that someone edit) the spreadsheet to reflect any new or changed analytics.

@danielebogo danielebogo added this to the 7.97 milestone Aug 19, 2025
@danielebogo danielebogo requested a review from a team as a code owner August 19, 2025 17:06
@danielebogo danielebogo requested review from bjtitus and removed request for a team August 19, 2025 17:06
Base automatically changed from danieleb/Playlists/PCIOS25/PCIOS-46-93-episodes to trunk August 20, 2025 13:30
Copy link
Contributor

@SergioEstevao SergioEstevao left a comment

Choose a reason for hiding this comment

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

:shipit:

@danielebogo danielebogo merged commit 7c61fa8 into trunk Aug 22, 2025
8 of 10 checks passed
@danielebogo danielebogo deleted the danieleb/Playlists/PCIOS25/PCIOS-94-increase-limit branch August 22, 2025 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants