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

Email notifier is not saving emails to db #7459

Closed
nobodyspecial opened this issue Dec 7, 2019 · 11 comments
Closed

Email notifier is not saving emails to db #7459

nobodyspecial opened this issue Dec 7, 2019 · 11 comments

Comments

@nobodyspecial
Copy link

Describe the bug
Show notification list in email notifier is not saving emails to database.

To Reproduce
Steps to reproduce the behavior:

  1. Go to Notifications -> Social -> Email
  2. Click on Select a show -> Any show -> Add email -> Save
  3. Email appears to save
  4. Emails not sent after snatch or post-process

Expected behavior
Emails should save to db

Screenshots
medusa

Medusa (please complete the following information):

  • OS: Windows-2003Server-5.2.3790-SP2
  • Branch: master, develop
  • Commit: fcb5a19
  • Python version: 2.7.10
  • Database version: 44.14

Debug logs (at least 50 lines):
General > Advanced Settings > Enable debug

2019-12-07 07:47:18 DEBUG	APIv2-Thread-1 :: Broken providers already updated in the last hour

Additional context

When saving an email, the above message appears in the debug log. I have checked the main.db file and confirmed notify_list is empty for all shows in the tv_shows table after saving. Also, the highlighted text in the image is wrong but unrelated to this issue I think.

@triage-new-issues triage-new-issues bot added the triage Untriaged issue label Dec 7, 2019
@medariox medariox added Support and removed triage Untriaged issue labels Dec 8, 2019
@medariox
Copy link
Contributor

medariox commented Dec 8, 2019

You need to click on the + and save afterwards.

@nobodyspecial
Copy link
Author

You need to click on the + and save afterwards.

Yes, I know this. It is only saving on the frontend and not in db.

@medariox
Copy link
Contributor

medariox commented Dec 8, 2019

What do you mean by that? The settings are saved in config.ini, not in the DB.

@nobodyspecial
Copy link
Author

What do you mean by that? The settings are saved in config.ini, not in the DB.

Are you sure the individual show notification emails are saved in config.ini? I know the rest of the email settings are saved there and those work fine.

@p0psicles
Copy link
Contributor

I believe those are saved in a table. But I have to double check

@nobodyspecial
Copy link
Author

notify_list in tv_shows table

@p0psicles
Copy link
Contributor

Yes. Like prowl and email is stored there

@nobodyspecial
Copy link
Author

Can anyone confirm this bug or is there something wrong with my install?

@chaddoncooper
Copy link

I am also seeing the issue, the individual show's email addresses are being sent correctly in a separate payload but they aren't persisted to the db. I am clicking the plus after adding the show.
Another area using a similar mechanism - required words saves to the is_required_word field just fine.

@nobodyspecial
Copy link
Author

nobodyspecial commented Sep 23, 2020

What more info is needed on this? I'm now running this in a docker in unraid and it's still not fixed. In fact now the "Select a show" drop down is missing now.

@p0psicles p0psicles self-assigned this Sep 24, 2020
@p0psicles
Copy link
Contributor

Fixed in #8535

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants