cocodelabs / znc-palaver

ZNC module for push notifications
MIT License
112 stars 14 forks source link

Add setting to allow notifications while connected. #4

Open fsargent opened 11 years ago

fsargent commented 11 years ago

As a user, I sometimes want to be notified that I've been mentioned while a client is still connected to ZNC, so that I can leave my connection open, and still get notified.

dewey commented 11 years ago

I'd be interested in this as well.

bizmurr commented 9 years ago

This would be fantastic, rather than expecting all clients to be disconnected/away maybe it should be all palaver clients?

kylef commented 9 years ago

@bizmurr The whole idea is that while you're at your computer on IRC, Palaver won't send you push notifications.

Ideally, you would use the clientaway module and configure your client to auto-away when you're not there. That way, while your on IRC you will not receive notifications elsewhere, and when you leave your computer or disconnect from IRC, you will start receiving notifications.

omnicons commented 9 years ago

Well yes, but see here is that there's plenty of people who'd like to be notified regardless. Colloquy has no problem adding an extra setting for this with its ZNC module. A little extra configuration would be nice that way users can have it work tailored to their workflow with IRC rather than needing all to conform to how you'd like to see it be done. Everyone uses ZNC a little differently and there are a ton of different ways to configure stuff. I personally use weechat and ZNC at the same time as a bouncer, other people might just use ZNC to make it so that their phone isn't making other people in a channel mad with disconnects.

I do see your point however with clientaway. It's just not everyone is looking to set up a bunch of modules and use them, and is looking for the app and its associated plugin for the configuration than going elsewhere for it.

farcaller commented 9 years ago

+1 to this. In a perfect world I'd prefer this to be configurable by something like /msg *palaver always/auto/never.

OstlerDev commented 9 years ago

+1 :+1:

115100 commented 5 years ago

https://github.com/115100/znc-palaver/commit/a0527e75996b663ba46167a084dbcd17229998ff

This should work.

rexbinary commented 3 years ago

I leave multiple computers on all the time connected to znc. It would be nice if I could be notified while connected when I am not at my computers. Then I would not have to kill all my irc clients when I leave just to get notified, and then relaunch them when I return.

AntiCompositeNumber commented 3 years ago

I previously used another client with znc notification support that would send notifications when /away was used, without the requirement for clientaway. That seems better to me than requiring an external module that the author considers deprecated.

beren12 commented 3 years ago

I agree, I would love a setting to be able to get notifications always, or if just there are no palaver clients active, or there are no clients at all active. Bonus points if you could have an idle timer, like 1min, 5min, etc so even if palaver is open on my iPad, if I haven't sent a message in the last few mins I'll get a notification.

beren12 commented 3 years ago

@bizmurr The whole idea is that while you're at your computer on IRC, Palaver won't send you push notifications.

Ideally, you would use the clientaway module and configure your client to auto-away when you're not there. That way, while your on IRC you will not receive notifications elsewhere, and when you leave your computer or disconnect from IRC, you will start receiving notifications.

Not all clients can auto-away. Sad, but true. I'm looking at you HexChat, LimeChat…

My workaround is to change if (m_pNetwork->IsUserOnline() == false) { to if (true) {

iNoNaMouSe commented 1 year ago

I agree. I really need this option! I would love to get nick highlight/message notifications even if my other client is still connected. It can just be a simple on/off setting that could be changed. Please add!

voaii commented 1 year ago

115100@a0527e7

This should work.

Thanks SO MUCH!