-
-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add an option to use notice instead of privmsg. #79
Comments
This would be nice, but probably never happens as most of people ues stupid IRC clients which put notices to random buffers/windows. |
There is a difference between a |
They should, but they don't. |
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. |
Please see TkTech/notifico#79 * > 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. ~~ @Dav1dde * > They should, but they don't. ~~ @Mkaysi * > 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. ~~ @sckasturi
@Mkaysi Do you really have to spam this issue everywhere? |
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. |
But you don't have to reference it in both commits and issues. |
commit ~= pull request when you use the GitHub web interface and this is entirely unrelated to the actual issue. |
I mentioned it because it spams the comment section here. But we should stop before our comments cause more spam than the references. |
Please see TkTech/notifico#79 * > 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. ~~ @Dav1dde * > They should, but they don't. ~~ @Mkaysi * > 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. ~~ @sckasturi
The RFC says:
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 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. |
Please add an option to use
NOTICE
instead ofPRIVMSG
. For example, Github has a little checkbox for this option: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.
The text was updated successfully, but these errors were encountered: