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

Fix "Enable notification emails" #609

Merged
merged 1 commit into from
Jul 7, 2021

Conversation

nickvergessen
Copy link
Member

Signed-off-by: Joas Schilling [email protected]

@nickvergessen
Copy link
Member Author

/backport to stable22

@nickvergessen
Copy link
Member Author

/backport to stable21

@nickvergessen
Copy link
Member Author

/backport to stable20

@artonge
Copy link
Collaborator

artonge commented Jul 7, 2021

Should get fixed by this one too: #589

@artonge
Copy link
Collaborator

artonge commented Jul 7, 2021

But backport is nice

@NextClaus
Copy link

When do you fix this annoying bug?

  • Notification checkbox cannot be enabled again in GUI (but only on database level)
  • Notification mails (if there are file changes in a folder) don't get sent

@nickvergessen
Copy link
Member Author

Feel free to send a Pull request and I will test and merge it by the end of the day

@NextClaus
Copy link

A "Pull request" seems to be a kind of suggestion for a code change. I'm not a coder at all. So I cannot contribute anything than telling you that the change which was implemented here in #609 doesn't seem to work. But "sandro-tassa" added a comment on 12 Oct. Maybe this helps fixing it?

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

Successfully merging this pull request may close these issues.

4 participants