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 notification delay should have an option to be shortened #6306

Closed
angerized opened this issue Aug 30, 2017 · 2 comments
Closed

Email notification delay should have an option to be shortened #6306

angerized opened this issue Aug 30, 2017 · 2 comments

Comments

@angerized
Copy link

angerized commented Aug 30, 2017

Steps to reproduce

  1. Create a share with another user
  2. Recipient is informed of this way too late

Expected behaviour

At the very least shorten the notification delay (maybe add a 15 minutes option).
Idealy recipients should be informed instantly.

Actual behaviour

Recipient is informed over an hour after the share has been established.

Server configuration

Debian 9 64 bits
Nextcloud 12.0.2 fresh install
MariaDB 10.1.26
PHP 7

@angerized
Copy link
Author

angerized commented Aug 30, 2017

On this page @Rello suggested to open a pull request with other options. I'm not sure how to do this. But I suppose the code could look something like that:

email_batch_time = 3600;
if ($notify_setting_batchtime === UserSettings::EMAIL_SEND_FIFTEENMINUTES) {
$email_batch_time = 900;
} else if ($notify_setting_batchtime === UserSettings::EMAIL_SEND_DAILY) {
$email_batch_time = 3600 * 24;
} else if ($notify_setting_batchtime === UserSettings::EMAIL_SEND_WEEKLY) {
$email_batch_time = 3600 * 24 * 7;
}

@MorrisJobke
Copy link
Member

This will be fixed in 12.0.3 There will be a dedicated sharing email.

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

No branches or pull requests

2 participants