Skip to content
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

Cannot @mention Matrix users in Discord #760

Closed
tgr opened this issue Dec 29, 2021 · 4 comments · Fixed by #829
Closed

Cannot @mention Matrix users in Discord #760

tgr opened this issue Dec 29, 2021 · 4 comments · Fixed by #829
Labels
T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems.

Comments

@tgr
Copy link

tgr commented Dec 29, 2021

While Discord mentions are correctly transformed into Matrix mentions, in Discord I can only @mention Discord users, not Matrix users. Makes sense since presence is not synced, but mentions (ie. sending notifications) are a very fundamental feature of any chat system so those not working degrades the user experience a lot. IMO some poor-man's mention support would be better than nothing - just check for any plaintext @<matrix-nick> string in Discord messages and convert to a mention if found.

@Zwelf
Copy link

Zwelf commented Jan 10, 2022

For me this is working. Both @<matrix-nick> and just <matrix-nick> results in a ping and notification and red message highlighting. Using element-web (not sure if this is relevant). Example with nick the display name of the matrix user:

discord matrix
discord-mention matrix-mention

@tgr
Copy link
Author

tgr commented Jan 10, 2022

You are right about notifications since those are just simple text matches. But the nick does not get recognized. Otherwise it would look like this:
matrix-nick-bubble
But given that as you say this doesn't really affect notifications, maybe it doesn't matter much in practice, especially given there is no easy or elegant fix...

@Zwelf
Copy link

Zwelf commented Jan 11, 2022

Yeah, I now see what you are referencing. Probably solvable for the common case, when each username is only given once. But there might not be a solution when multiple users have the same name?

@deepadmax
Copy link

It could at least parse @username:server.tld into [](https://matrix.to/#/@username:server.tld), albeit it's more effort to enter.

@jaller94 jaller94 added the T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems. label May 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants