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.

Sign up for GitHub

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

Jump to bottom

Email Notification Settings do not save #251

Closed
loheiman opened this issue Dec 14, 2020 · 3 comments
Closed

Email Notification Settings do not save #251

loheiman opened this issue Dec 14, 2020 · 3 comments

Comments

@loheiman
Copy link

loheiman commented Dec 14, 2020 •
edited

on the :develop channel

I copied the email notification settings over from Ombi and when I click "Save", nothing happens. No notification and I remain on the page. When I navigate away and then back to email settings, the settings I entered are no longer present. I've confirmed the setting in Ombi work by sending myself a test email.

Screen Shot 2020-12-14 at 12 14 11 AM
@sct
Copy link
Owner

sct commented Dec 14, 2020

I have an idea why this might be happening. Probably the same reason the ports for radarr/sonarr settings were failing. Let me try to get out a fix for this!

sct added a commit that referenced this issue Dec 14, 2020
@sct
fix(frontend): converts email smtp port to a number before posting to…
2098a2d 
… the api

- also adds toast notifications to both email/discord notifications pages to know when sucesses or
failures occur

re #251
@sct
Copy link
Owner

sct commented Dec 14, 2020

The latest develop tag should have resolved this. Let me know if you are still having trouble!

@sct sct closed this as completed Dec 14, 2020
@loheiman
Copy link
Author

Confirmed that this is now fixed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sct @loheiman