-
-
Notifications
You must be signed in to change notification settings - Fork 106
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 option to the NotificationHandler and option to extend the msg #211
Comments
Not sure I get the use case |
To enrich the log message, I'd like to have more information added on top of the log message when sending the notification. |
Looking into this now. Seems it would be kinda tricky to define this in a generic way, since each developer has different body. Would you add it to email subject? To push notification as a prefix/suffix? Seems like this should be handled by the notification sender |
I'd llike to get my hands dirty on it, |
Add an option to send additional data in the notification besides the log msg (e.g. custom msg)
example:
notification:
The text was updated successfully, but these errors were encountered: