TkTech / notifico

IRC Message Relay
http://n.tkte.ch
MIT License
155 stars 39 forks source link

Add an option to use notice instead of privmsg. #79

Open ghost opened 10 years ago

ghost commented 10 years ago

Please add an option to use NOTICE instead of PRIVMSG. For example, Github has a little checkbox for this option: 2014-05-31-104637_161x43_scrot

Many IRC clients allow users to disable notifications (beeps, etc) for notices. This is often useful in channels where notifico messages are considered of much lesser importance than messages from users.

Mikaela commented 10 years ago

This would be nice, but probably never happens as most of people ues stupid IRC clients which put notices to random buffers/windows.

Dav1dde commented 10 years ago

There is a difference between a NOTICE to a channel and a NOTICE to a user, every client should be able to correctly display notices to a channel.

Mikaela commented 10 years ago

They should, but they don't.

sckasturi commented 10 years ago

That's besides the point. There are a lot of clients that do handle it properly, and if they don't then that's not notifico's fault, that's their own fault.

ghost commented 10 years ago

@Mkaysi Do you really have to spam this issue everywhere?

Mikaela commented 10 years ago

If I didn't specify my source, it was questioned and references to issue don't send automatic emails unlike useless comments that are unrelated to the actual issue.

ghost commented 10 years ago

But you don't have to reference it in both commits and issues.

Mikaela commented 10 years ago

commit ~= pull request when you use the GitHub web interface and this is entirely unrelated to the actual issue.

ghost commented 10 years ago

I mentioned it because it spams the comment section here. But we should stop before our comments cause more spam than the references.

Mikaela commented 7 years ago

The RFC says:

4.4.2 Notice

      Command: NOTICE
   Parameters: <nickname> <text>

   The NOTICE message is used similarly to PRIVMSG.  The difference
   between NOTICE and PRIVMSG is that automatic replies must never be
   sent in response to a NOTICE message.  This rule applies to servers
   too - they must not send any error reply back to the client on
   receipt of a notice.  The object of this rule is to avoid loops
   between a client automatically sending something in response to
   something it received.  This is typically used by automatons (clients
   with either an AI or other interactive program controlling their
   actions) which are always seen to be replying lest they end up in a
   loop with another automaton.

   See PRIVMSG for more details on replies and examples.

The difference between NOTICE and PRIVMSG is that automatic replies must never be sent in response to a NOTICE message.

As said before, this isn't the case with IRC in practice (thanks mIRC), but nowadays there is also Matrix which doesn't have this issue.

Matrix uses m.notice for messages by bots (like was supposed to be done with IRC) and native clients (mainly Riot) have option to not highlight on messages by bots thanks to it.

How this is related to IRC commit announcer bot is that Matrix has bridges to multiple IRC networks (Bridged IRC networks @ matrix-org/matrix-appservice-irc wiki (can be thought of as a stupid bouncer)) and thus the users encounter notifico and committetters/authors get pinged by it. If Notifico was able to use NOTICE, that didn't happen as the bridge turns NOTICE to m.notice.

In addition to having this option, there should be a way to mass edit channels or at least edit channels. I would be happy with option to make all channels under my account to use NOTICEs or at least default to it unless I otherwise specified.