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

Next show data missing for a suivre because we only expose the next 60d #21

Open
hairmare opened this issue Jan 29, 2025 · 0 comments
Open
Assignees

Comments

@hairmare
Copy link
Member

hairmare commented Jan 29, 2025

Since we currently only return the schedule for the next 60 days from libretime, shows with less frequent episodes don't look nice on the page:

Image

This affects other shows as well, but this analysis is based on a suivre. Initially we settled on 60d of "next" data because there were often errors in the schedule further out.

To support more use cases, we should return the schedule for a much longer time period from libretime. A range that would fix a suivre and then some would be to return the next 6 months (183d).

Doing this would increase the size of the returned JSON considerably. The current result is 156K, the new one would be 428K. Storage wise this isn't much, but it feels like the increase in size might have a notable impact on page loading times.

If my concern is valid, then we might need to explore other options, like splitting the file into monthly chunks.

I'll try to test this, most likely over the upcoming weekend, and either update this issue to reflect my findings or close it if the fix works.

@hairmare hairmare self-assigned this Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant