Add on_dispatched event

[PhrozenByte]

This event is fired when a notification was sent to the notifications server. This is less useful as class-level callback, but very useful with Notification, because it allows for a better separation of concerns (separates creating and sending notifications).

ToDo:

• After merging Quality of life improvements to Notification class #203 Notification.on_dispatched must be changed to:

      on_dispatched: Callable[[], Any] | None = field(default=None, repr=False)

• After merging DesktopNotifierBackend: Drop notification param from handle_*() methods #205 DesktopNotifierBackend.on_dispatched must be changed to the following (also update the caller accordingly):

      def handle_dispatched(self, identifier: str) -> None:
          if identifier in self._notification_cache:
              notification = self._notification_cache[identifier]
              if notification.on_dispatched:
                  notification.on_dispatched()
                  return
          if self.on_dispatched:
              self.on_dispatched(identifier)

• Add on_dispatched to DesktopNotifierSync after merging DesktopNotifierSync: Add on_* properties for class-level events #207

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 94.11765% with 1 line in your changes missing coverage. Please review.

Project coverage is 85.23%. Comparing base (26c6af2) to head (fc660b2).
Report is 23 commits behind head on main.

Files with missing lines	Patch %	Lines
src/desktop_notifier/main.py	83.33%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #209      +/-   ##
==========================================
+ Coverage   85.13%   85.23%   +0.09%     
==========================================
  Files          10       10              
  Lines         895      921      +26     
==========================================
+ Hits          762      785      +23     
- Misses        133      136       +3     

Flag	Coverage Δ
pytest	85.23% <94.11%> (+0.09%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[samschott]

Thanks, I can see this being useful! Could you check if similar events are available on macOS and Windows?

[PhrozenByte]

This is a Desktop-Notifier-only event, i.e. doesn't rely on the notifications server and thus works with any backend.

[samschott]

Makes sense! I confused this with an event for notification being delivered, which would indeed need some information from the platforms.