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

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

Closed
sephrat opened this issue Dec 17, 2021 · 3 comments · Fixed by #4627
Closed

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

sephrat opened this issue Dec 17, 2021 · 3 comments · Fixed by #4627

Comments

@sephrat
Copy link
Contributor

sephrat commented Dec 17, 2021

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

  • The block shouldn't display at all instead of showing without any item.
  • One item should have popped in the list. The lead I have so far: it was added to Emby a few hours before the newsletter job, but is not referenced as "Available" in Ombi yet. Maybe there's an issue with the newsletter when Emby and Ombi are out of sync?
    Edit: actually there's a second item that should have popped and is properly synced in Ombi, so that's not it. Will try to find more useful info.

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):

  • Version 4.7.8
  • Media Server : Emby
@github-actions
Copy link

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
Copy link
Contributor Author

sephrat commented Dec 17, 2021

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.

var info = await _tvApi.ShowLookupByTheTvDbId(tvdbId);
if (info == null)
{
continue;
}

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
Copy link
Member

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

tidusjar pushed a commit that referenced this issue Feb 1, 2022
* Stop using TVMaze in newsletter

* Better logging when an item is not published
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants