jamulus icon indicating copy to clipboard operation
jamulus copied to clipboard

Event/Growl notifications on client desktop machines

Open corrados opened this issue 5 years ago • 7 comments

Allow Jamulus optionally to send visual event notifications using the host OS's notification function.

Examples including but not necessarily limited to:

  • Player entering/leaving server
  • Chat message notifications

Which events (or event types) to show should be configurable in Settings.

corrados avatar Jul 03 '20 14:07 corrados

Is this ticket about event notifications in general? I can see a use case for an OS-based notifications option. For example, "server recording has been turned on", "[name] sent a chat message", or even "Delay has been too high for too long".

gilgongo avatar Jul 05 '20 08:07 gilgongo

With >=Qt5.6 (I understand), qInfo(), etc use system logging, so server events captured that way are covered. If you're running Jamulus Server on Linux, built with >=Qt5.6 and running under systemctl, for example, you should be able to run journalctl -f -u jamulus-server (or whatever your unit name is) to tail the messages. This is also capturing client connects and server stop messages.

So that raises potential concerns over how much GDPR impact there would be if the mechanism you used for notifications also allowed logging.

There's a separate Issue about debugging logs, too. It can be very useful to say "OK, try that again but run with --debug logfile.txt", rather than having someone try to explain how to recreate a problem. Assuming you've added all the necessary debug code...

pljones avatar Jul 05 '20 11:07 pljones

So that raises potential concerns over how much GDPR impact

In practice, there would be none relating to IP addresses, although it would help if Jamulus let server operators conform to the spirit of that law by notifying their users of the issues relating to IPs and names, as per #367

I think this ticket is more about event notifications on client desktop machines though, no?

gilgongo avatar Jul 05 '20 12:07 gilgongo

I think this ticket is more about event notifications on client desktop machines though, no?

Good question. I would say yes.

I just ported the Issue from Sourceforge. I am not sure if the originator of the feature requests reads this forum here at Github.

corrados avatar Jul 05 '20 15:07 corrados

Editing the ticket to summarise it better. If anyone wants to pick it up and run with it, please do so.

gilgongo avatar Feb 19 '21 13:02 gilgongo

Seems like this has been addressed by https://github.com/jamulussoftware/jamulus/pull/2640 so I'll close.

gilgongo avatar Jul 13 '22 21:07 gilgongo

#2640 uses audio notifications. This issue is for OS popups. The two aren't particularly related.

pljones avatar Jul 14 '22 06:07 pljones