[Feat]: should the DefaultRequestHandler.on_message_send() method fire push notifications? #213
Description
Is your feature request related to a problem? Please describe.
currently it just added the push notification config to the task and the only way to trigger push notifications is through on_message_send_stream() method
Describe the solution you'd like
Should both on_message_send_*() methods start a background task to fire the push notifications? or should there be a pushnotification service that picks up tasks from the event queue and fires push notifications?
@holtskinner in case you have any thoughts on this.
Describe alternatives you've considered
No response
Additional context
No response
Code of Conduct
- I agree to follow this project's Code of Conduct