matrix-org / matrix-appservice-irc

Node.js IRC bridge for Matrix
Apache License 2.0
462 stars 151 forks source link

"(IRC)" should only be part of the display, not of the message #1563

Closed pchampin closed 2 years ago

pchampin commented 2 years ago

Is your feature request related to a problem? Please describe. When I use auto-completion to mention an IRC user (say 'john'), my matrix client auto-completes it as 'john (IRC)'. While I don't mind it being displayed "john (IRC)" in Matrix (it is an interesting info), I don't think that this precision should be part of the actual message sent to IRC.

Describe the solution you'd like Either use "(IRC)" only in the Matrix client, but not in the messages sent to IRC, or (if it is too complex), drop the "(IRC)" mention all together (or make it configurable).

Mikaela commented 2 years ago

Which bridge and client are you seeing this with? It may be an issue with that particular instance as the "(IRC)" suffix was dropped back in August 2018 https://github.com/matrix-org/matrix-appservice-irc/issues/194#issuecomment-415734087 (and before that in 2016 "Riot" used to remove it automatically https://github.com/matrix-org/matrix-appservice-irc/issues/194#issuecomment-268208287).

pchampin commented 2 years ago

the "(IRC)" suffix was dropped back in August 2018 Was it? Interesting :thinking:

I am using Element Desktop for Linux: Element version: 1.10.11 Olm version: 3.2.8

My server is matrix.org. An example IRC channel I use is https://matrix.to/#/#_w3c_#rdf-dev:matrix.org

tadzik commented 2 years ago

This is indeed a misconfiguration of w3c bridge on matrix org, due to the fact that this is the IRC bridge's default behaviour. Dropping it from the example config is probably a sensible thing to default to.

pchampin commented 2 years ago

Is it something that I should discuss with the people at Matrix.org? The people at W3C?

tadzik commented 2 years ago

That's something that we, as a team maintaining matrix.org bridges, need to do – shouldn't be long now :)