matrix-org / matrix-org/mjolnir
"Applying ACL in <room>" and other spoilered room IDs should use pills/aliases where possible
- Dominant language
- TypeScript
- Stars
- 445
- Forks
- 75
- Avg merge
- 2d 44m
- Merged PRs (30d)
- 3
Description
it's more user friendly to see what room it is. If we can't trick element to spoiler a pill (ie: putting a matrix.to link under the spoiler), include the room ID next to the alias.
For example:
```
Applying ACL in #room:example.org (!room:example.org)
```
If there's no alias, fall back to just room ID (as today).
Contributor guide
Research direction
Search the TypeScript sources for the exact “Applying ACL in” message and the code that formats room IDs. Check how Matrix room aliases and pills are currently represented, then make the output prefer a room alias with the room ID alongside it, falling back to the ID when no alias exists. Confirm the resulting message matches the example and preserves spoiler behavior where supported.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 42/100