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

Updated spatie backup notification config class names #14877

Merged
merged 1 commit into from
Jun 17, 2024

Conversation

snipe
Copy link
Owner

@snipe snipe commented Jun 17, 2024

Looks like Spatie changed some class names that we missed in the config. This should correct those method names.

Signed-off-by: snipe <[email protected]>
Copy link

what-the-diff bot commented Jun 17, 2024

PR Summary

  • Updates to Notification Classes
    We have updated the references in our backup configuration file with new class definitions for notification handling. This change aligns each status event (backup failure, unhealthy backup discovery, cleanup failure, successful backup, healthy backup discovery, and successful cleanup) to its corresponding notification class. These updates refine how the system communicates backup states, resulting in clearer, more consistent notifications.

@snipe snipe merged commit afc7852 into develop Jun 17, 2024
7 of 8 checks passed
@snipe snipe deleted the fixes/spatie-backup-config branch June 17, 2024 12:43
@snipe snipe changed the title Updated config Updated spate backup config class names Jun 17, 2024
@snipe snipe changed the title Updated spate backup config class names Updated spatie backup config class names Jun 17, 2024
@snipe snipe changed the title Updated spatie backup config class names Updated spatie backup notification config class names Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant