Open acagastya opened 7 years ago
I think this is fixed by https://github.com/FruitieX/teleirc/commit/eb1b4652f221168321d7faa4ed2f08fe883a2c00 ?
Nick changes are not relayed, yet.
@acagastya Which version of TeleIRC are you using? Does it include that comment, could you try to do the development install?
See CONTRIBUTING.md for instructions.
Yes, it contains an array of IRC events, join and part is working, but nick changes are not relayed.
Giving my 2 cents here: not a bug, but a enhancement.
nodeIrc.on('nick', function(event) {
if (!shouldRelayEvent('nick')) {
return;
}
var chanName = event.channel;
var user = event.nick;
var newNick = event.new_nick;
var channel = ircUtil.lookupChannel(chanName, config.channels);
msgCallback({
protocol: 'irc',
type: 'nick',
channel: channel,
user: null,
text: user + ' is now known as ' + newNick
});
});
Something like this would not work because event.channel is undefined
.
We'd have to send this notice to all chans where the user is.
One way to achieve this would be to use node-irc instead of irc-framework. This might need some code rewriting, I still need to investigate how bad that would be.
Edit: Seems like that would not necessarily be a good idea (c.f. https://github.com/FruitieX/teleirc/commit/4f26402e966a3e0f23b9b64facbc1610af0cbf1b)
config.sendNonMsg = true;
sends IRC joins, parts and quits to Telegram -- but it won't inform if the user changes their nick. If I change the nick, that information should be relayed as well.Anyone who joined the channel, changed the nick, and left should be announced on Telegram group just like how IRC announces it in the chatrooms.