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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Push rules do not get copied to the new room when it is upgraded #2863

Closed
ShadowJonathan opened this issue Nov 5, 2022 · 1 comment · Fixed by #2864
Closed

Push rules do not get copied to the new room when it is upgraded #2863

ShadowJonathan opened this issue Nov 5, 2022 · 1 comment · Fixed by #2864

Comments

@ShadowJonathan
Copy link

This fails a sytest called $user_type user has push rules copied to upgraded room

It asserts that push rules are copied to the new room when it has been upgraded (and the user joins the new room)

See the following wording, where the spec recommends this:

When a user joins the new room, the server should automatically transfer/replicate some of the user’s personalized settings such as notifications, tags, etc.

Background information

  • Dendrite version or git SHA: b13cb43
@ShadowJonathan
Copy link
Author

(Whoops)

@S7evinK highlighted for me that Dendrite does indeed pass the sytest, but after some research, I found out that the sytest is very likely faulty.

Furthermore, when scouring the dendrite codebase, I could not find a single ounce of behaviour (searching on strings "upgrade" and "tombstone") that referred to an after-upgrade personal user settings migration, if that user joins such an upgraded room.

S7evinK added a commit that referenced this issue Nov 7, 2022
Fixes #2863 and makes 
```
/upgrade preserves direct room state
local user has tags copied to the new room
remote user has tags copied to the new room
```
pass.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant