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

Add CKUT outstanding issues #3

Closed
3 tasks done
shawnCaza opened this issue Jun 18, 2024 · 3 comments · Fixed by #4
Closed
3 tasks done

Add CKUT outstanding issues #3

shawnCaza opened this issue Jun 18, 2024 · 3 comments · Fixed by #4

Comments

@shawnCaza
Copy link
Owner

shawnCaza commented Jun 18, 2024

Based on a request here, I have added CKUT podcast feeds to the site. Some outstanding issues that arose in the process:

  • Search behaves in unexpected ways when certain search terms are used. EX. searching for 'am' produces multiple results for a single show, and these results are not fully cleared when the search text is deleted.

  • The date of the added show doesn't include a time. Unexpected difficulty getting the modified date from the mp3 header.

  • In the apple podcast app, episodes do not always successfully play on the first attempt.

@shawnCaza
Copy link
Owner Author

Solved the date issue, by parsing it out of the mp3 file name.

Not playing on first attempt was due to multiple redirects on the mp3 download link.

@shawnCaza
Copy link
Owner Author

Search issues fixed by re-scraping episodes for two shows where duplicate newest ep_date were present.

@shawnCaza
Copy link
Owner Author

Add an index to the episodes table in the DB to enforce show_id and ep_date as unique value pairs. This will eliminate the possibility of a show with multiple episodes sharing the same date and time.

@shawnCaza shawnCaza linked a pull request Jun 21, 2024 that will close this issue
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

Successfully merging a pull request may close this issue.

1 participant