Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Remote aliases aren't moved to new room on room upgrade. #6784

Closed
mvgorcum opened this issue Jan 26, 2020 · 7 comments
Closed

Remote aliases aren't moved to new room on room upgrade. #6784

mvgorcum opened this issue Jan 26, 2020 · 7 comments
Assignees
Labels
A-Room-Upgrades z-p2 (Deprecated Label)

Comments

@mvgorcum
Copy link
Contributor

Description:

@anoadragon453 anoadragon453 self-assigned this Jan 26, 2020
@richvdh richvdh changed the title Remote aliasses aren't moved to new room on room upgrade. Remote aliases aren't moved to new room on room upgrade. Jan 30, 2020
@richvdh
Copy link
Member

richvdh commented Jan 30, 2020

Could someone explain like I'm five what exactly the problem is here?

@mvgorcum
Copy link
Contributor Author

I haven't tested all permutations, but this is what I found:

@exampleuser:server1.com creates room, adds alias #alias:server1.com. @exampleuser2:server2.com joins and adds #alias2:server2.com.

@exampleuser:server1.com upgrades the room to another room version. #alias:server1.com gets moved to the new room, but #alias2:server2.com does not get moved.

@anoadragon453
Copy link
Member

@mvgorcum Does the second alias get moved when a user from server2.com joins the upgraded room?

@anoadragon453
Copy link
Member

Just tested and indeed remote aliases don't ever get moved. We should move them when the server processes the create event for the first time, just like other room attributes.

@mvgorcum
Copy link
Contributor Author

mvgorcum commented Jan 30, 2020

Yeah, if @exampleuser2:server2.com joins that doesn't trigger an alias move.

Things I haven't tested include: what if @exampleuser2:server2.com upgrades the room, or if @exampleuser3:server3.com doesn't set an alias, but does upgrade the room? etc.

Also: apologies for not spelling it out properly in the first place

@neilisfragile neilisfragile added the z-p2 (Deprecated Label) label Feb 5, 2020
@richvdh
Copy link
Member

richvdh commented Feb 5, 2020

related: #6842

@anoadragon453
Copy link
Member

Fixed by #6946

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Room-Upgrades z-p2 (Deprecated Label)
Projects
None yet
Development

No branches or pull requests

4 participants