Ombi-app / Ombi

Want a Movie or TV Show on Plex/Emby/Jellyfin? Use Ombi!
http://ombi.io
GNU General Public License v2.0
3.7k stars 395 forks source link

Newsletter sent with a "New TV" block but no item below #4444

Closed sephrat closed 2 years ago

sephrat commented 2 years ago

Describe the bug Just received my weekly newsletter with a "New TV" block without any item below. I think there are two inherent issues:

Screenshots image

Logs (Logs directory where Ombi is located)

2021-12-17 12:00:00.001 +01:00 [Debug] Calling Execute on job System.INewsletterJob
2021-12-17 12:00:00.005 +01:00 [Debug] Trigger instruction : NoInstruction
2021-12-17 12:00:17.923 +01:00 [Error] StatusCode: NotFound, Reason: Not Found, RequestUri: http://api.themoviedb.org/3/find/?api_key=b8eabaf5608b88d0298aa189dd90bf00&external_source=imdb_id
2021-12-17 12:00:19.432 +01:00 [Error] StatusCode: NotFound, Reason: Not Found, RequestUri: http://api.tvmaze.com/lookup/shows?thetvdb=82890
2021-12-17 12:00:19.480 +01:00 [Error] StatusCode: NotFound, Reason: Not Found, RequestUri: http://api.tvmaze.com/lookup/shows?thetvdb=412366

Ombi Version (please complete the following information):

github-actions[bot] commented 2 years ago

Hi!
Thanks for the issue report. Before a real human comes by, please make sure you used our bug report format.
Have you looked at the wiki yet? https://docs.ombi.app/
Before posting make sure you also read our FAQ.
Make the title describe your issue. Having 'not working' or 'I get this bug' for 100 issues, isn't really helpful.
If we need more information or there is some progress we tag the issue or update the tag and keep you updated.
Thanks!
Ombi Bot.

sephrat commented 2 years ago

Ah, I think I see it. The newsletter generates the "New TV" block when there are new episodes but the individual episodes won't show if no info could be found from TVMaze API. https://github.com/Ombi-app/Ombi/blob/5e140ab6183b887a7665f5e870eb0bd05d487ace/src/Ombi.Schedule/Jobs/Ombi/NewsletterJob.cs#L1069-L1073 We need to generate the "TV block" only when we know for sure some content will be generated after. On top of that, shouldn't we move from TVMaze to TMDB to be consistent with the rest of the app?

tidusjar commented 2 years ago

Looks like that's an area we missed migrating from from TvMaze