-
Notifications
You must be signed in to change notification settings - Fork 497
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
Fix inoperant room links with alias/identifiers #6396
Conversation
📱 Scan the QR code below to install the build for this PR. If you can't scan the QR code you can install the build via this link: https://i.diawi.com/b1vFVS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not against this change, but I've added a comment inline with a small query.
Codecov Report
@@ Coverage Diff @@
## develop vector-im/element-web#6396 +/- ##
===========================================
+ Coverage 6.22% 6.26% +0.04%
===========================================
Files 1451 1451
Lines 155597 155599 +2
Branches 62507 62509 +2
===========================================
+ Hits 9689 9756 +67
+ Misses 145504 145437 -67
- Partials 404 406 +2
Continue to review full report at Codecov.
|
Kudos, SonarCloud Quality Gate passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that seems like a more sensible place to handle the error to me.
I haven't tested this PR but it appears to linkify room IDs. FWIW Element Web intentionally does not linkify room IDs |
Thanks for raising this @aaronraimist As a matter of fact these we're "always" linkified but the navigation stopped working even on servers that know or host the room. I raised a separate issue here: #6404 to actually remove the linkify. |
Fixes element-hq/element-web#6395