-
-
Notifications
You must be signed in to change notification settings - Fork 768
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
Mail notification is sent when it's not supposed to #1879
Comments
I can confirm this erratic behavior on my end 👍 |
@pxpm Could you take a look at this? If we don't find a fix for this soonish, I'll revert the PR. |
On it 👍 |
The same applies to the |
Fixed in the latest release. |
many thanks guys! |
Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
Since latest update (9.2.4) system is sending mail notification even if it doesn't suppose to.
Given this config:

CleanupWasSuccessfulNotification
shouldn't be sent but it is. I'm suspecting that it has something to do with the latest PR... specifically this line:https://github.com/spatie/laravel-backup/pull/1873/files#diff-18d9e9c6768a133f4fc7723d566dc36b5a1a07bee3c30396619c76cd85422bfeR31
The text was updated successfully, but these errors were encountered: