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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prefer using the canonical alias in spotlight search #9059

Merged
merged 1 commit into from Jul 15, 2022

Conversation

justjanne
Copy link
Contributor

@justjanne justjanne commented Jul 15, 2022

Type: Defect
Related: #9055


Here's what your changelog entry will look like:

馃悰 Bug Fixes

  • Prefer using the canonical alias in spotlight search (#9059). Contributed by @justjanne.

* Prefer using the canonical alias in spotlight search

Public rooms on other homeservers are not joinable via the roomId if they haven't been joined by other users on your homeserver.

* Ensure we call the action with the room_alias

* lint

* Drop display

* Always provide roomId

* Add rationale to room_id, room_alias

* whoops

Co-authored-by: Michael Telatynski <7t3chguy@gmail.com>
@justjanne justjanne added the T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems label Jul 15, 2022
@justjanne justjanne requested a review from a team as a code owner July 15, 2022 11:27
@justjanne justjanne requested review from andybalaam, t3chguy and weeman1337 and removed request for a team July 15, 2022 11:27
@t3chguy
Copy link
Member

t3chguy commented Jul 15, 2022

Test failure expected due to branch mismatch

Copy link
Contributor

@andybalaam andybalaam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No obvious (to me) errors

@t3chguy t3chguy merged commit 85363ef into release-v3.49.0 Jul 15, 2022
@t3chguy t3chguy deleted the justjanne/fix/spotlight-room-alias branch July 15, 2022 12:56
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 this pull request may close these issues.

None yet

4 participants