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

Add Gotify Notification Support #260

Merged
merged 1 commit into from
Aug 8, 2024
Merged

Conversation

MDeLuise
Copy link
Owner

@MDeLuise MDeLuise commented Aug 8, 2024

Hey Plant-it community!


This PR fixes #258, i.e. introduce a new feature that adds support for Gotify notifications. This addition will allow users who use Gotify for their notification needs to integrate seamlessly with the application.

What's new?

I have implemented Gotify notification support in the application. Users can now configure the application to send notifications via Gotify by setting up the appropriate server properties.

Why is it important?

Many users and organizations utilize Gotify for their notification systems. By adding support for Gotify, we provide more flexibility and options for our users to integrate their preferred notification services.

How to Use?

To enable Gotify notifications, you need to configure the new GOTIFY_ENABLED=true server property.



Cheers and happy planting! 🌿🌼

Copy link

github-actions bot commented Aug 8, 2024

📝 Coverage Report For Server Service

Overall Project 50.4% -1.86% 🟢
Files changed 10.69% 🔴

File Coverage
NotificationDispatcherName.java 100% 🟢
GotifyNotificationDispatcherDTOConverter.java 22.22% -77.78% 🔴
NotificationDispatcherController.java 13.19% -34.07% 🔴
GotifyNotificationDispatcherConfig.java 8.11% -91.89% 🔴
GotifyNotificationDispatcher.java 7.09% -92.91% 🔴
NtfyNotificationDispatcher.java 4.63% 🟢
GotifyNotificationDispatcherConfigDTO.java 0% 🔴

@MDeLuise MDeLuise merged commit 1e37f51 into main Aug 8, 2024
3 checks passed
@MDeLuise MDeLuise deleted the feature-gotifyNotification branch August 8, 2024 10:27
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

Successfully merging this pull request may close these issues.

Do you plan to add Gotify notification?
1 participant