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

Additional notification options #26

Closed
mentalsoup opened this issue Sep 12, 2022 · 5 comments · Fixed by #31
Closed

Additional notification options #26

mentalsoup opened this issue Sep 12, 2022 · 5 comments · Fixed by #31
Assignees

Comments

@mentalsoup
Copy link

What feature would you like added?

I would like to be able to send sync notifications for success/failure of orbital-sync to either/or:
Telegram chat group
Gotify server

Version

1.1.1 (latest)

Log output

No response

@mattwebbio
Copy link
Owner

Would the idea be to get success/failure notifications every interval a sync is performed?

If you use the default interval of 30 mins, this could generate a lot of noise. It might make more sense to only alert upon failure?

@mentalsoup
Copy link
Author

Fair comment, I agree that would get irritating. Alert on failure only by default perhaps. Then those that want the noise can manage it themselves.

@mattwebbio
Copy link
Owner

mattwebbio commented Sep 28, 2022

Sorry for the delay on this, I haven't had a lot of time and wanted to refactor some of the error handling to make handling notifications easier in the future.

I have a beta release for the container image that adds SMTP email notification support. The image has been published as: mattwebbio/orbital-sync:1.1.3-beta.2
The documentation for configuring it is available here: https://github.com/mattwebbio/orbital-sync/blob/7ec6205259731cdddd9184dcf6cda161e320f1b6/README.md#notifications

Do you feel support for other services such as Telegram or Gotify is important? I've not used either of those personally.

@mentalsoup
Copy link
Author

No issue on the timescale, take it at your own pace.
I do not consider either Telegram or Gotify essential. If you are happy with email notifications, then that's fine.
An alternative to direct integration with any system could be to execute a script if an error occurs, then you could leave it up to the user regarding their chosen method of notification.

@mattwebbio
Copy link
Owner

I've just merged and released SMTP as part of 1.2. Let me know if you have any issues with it :)

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 a pull request may close this issue.

2 participants