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

[stable21] Fix "Enable notification emails" #612

Merged
merged 1 commit into from
Jul 7, 2021

Conversation

backportbot-nextcloud[bot]
Copy link

backport of #609

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen merged commit 297e8d1 into stable21 Jul 7, 2021
@delete-merged-branch delete-merged-branch bot deleted the backport/609/stable21 branch July 7, 2021 13:09
@skjnldsv skjnldsv mentioned this pull request Jul 26, 2021
14 tasks
@skjnldsv skjnldsv mentioned this pull request Aug 3, 2021
12 tasks
@NextClaus
Copy link

Probably a stupid question: But where can I see a description of the actual issue which was supposed to be fixed?
Reason for asking: we upgraded to v21.0.4. However we are experiencing a bug which disables the central notification setting, respectively one cannot enable it again via WebGUI. Please have a look at the community post:

https://help.nextcloud.com/t/enable-notification-emails-activity-app-not-work/124699/5
https://help.nextcloud.com/t/activity-emails-not-sent/125597

@rakekniven
Copy link
Member

Hello @NextClaus , could not find a linked issue as well 😞

@nickvergessen
Copy link
Member

It came in via a customer report

The "Enable notification emails" check button in the Admin Settings -> Activity -> Notifications is not working properly. If it is via the occ command, it can be turned on and off.

And this here should fix that.

but yeah seems to not be the case anymore. Also see the comment in #609

@NextClaus
Copy link

Same here. 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

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.

3 participants