show "Room" for unresolved event permalinks

[Velin92]

This is the same behaviour of EW, that we want to have in EX for event permalinks

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[pixlwave]

LGTM 👌

[pixlwave]

Just to double check, do we just want to do this for events? I.e. should we also do the same for the roomID case?

[Velin92]

Just to double check, do we just want to do this for events? I.e. should we also do the same for the roomID case?

Checked with @Anderas and we want to have the same behaviour of EW, so... we apply the generic "Room" string only for event permalinks, room permalinks are already consistent with EW by showing the room ID

[pixlwave]

Makes sense 👍

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.96%. Comparing base (8906875) to head (7d2fc8f).
Report is 1 commits behind head on develop.

✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #3854   +/-   ##
========================================
  Coverage    78.95%   78.96%           
========================================
  Files          794      794           
  Lines        70221    70215    -6     
========================================
- Hits         55445    55442    -3     
+ Misses       14776    14773    -3     

Flag	Coverage Δ
unittests	70.85% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.