Xjph / ObservatoryCore

Elite Observatory: Extensible tool for processing Elite Dangerous journals
https://observatory.xjph.net/
MIT License
86 stars 16 forks source link

feat(core,framework): Allow plugins to opt in to batch-mode notifications #186

Closed fredjk-gh closed 2 weeks ago

fredjk-gh commented 2 weeks ago

Add a boolean property to the IObservatoryNotifier interface that allows a plugin to override the default suppression of sending notifications to plugins notifiers. This must be explicitly set by interested plugins so existing notifiers will not suddenly receive all the batch-read notifications.

This has the side benefit of cleaning up a couple bits of duplicate code wrapped in mutually exclusive #ifdef blocks.

This will enable some desireable functionality in Aggregator (showing notifications generated via pre-read and some testing via read-all) without negative impact to existing plugins (like telegram and StellarOverlay -- which presently spams me with notifications if I forget to disable it).