Skip to content

Support for MSC 4289#1908

Merged
Velin92 merged 6 commits intodevelopfrom
mauroromito/implement_new_MSC
Jul 23, 2025
Merged

Support for MSC 4289#1908
Velin92 merged 6 commits intodevelopfrom
mauroromito/implement_new_MSC

Conversation

@Velin92
Copy link
Copy Markdown
Member

@Velin92 Velin92 commented Jul 22, 2025

Room creator will now have the highest possible power level, which won't be determined by m.room.power_levels.
Also a room may have additional creators.

Also the powerLevelOfUserWithUserID function from MXPowerLevels has been removed in favour of the newer version in MXRoomState.

Velin92 added 4 commits July 22, 2025 15:00
- added the `additional_creators` field in the m.room.create event
- the creator is now determined as the sender of the m.room.create event
- power levels check for creator first and return the max possible PL in that case before checking the power levels event
Copy link
Copy Markdown
Member

@pixlwave pixlwave left a comment

Choose a reason for hiding this comment

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

LGTM (at least from what I understand about the MSC, I haven't read it fully yet).

Although note that the unit tests are failing to compile.

Copy link
Copy Markdown
Member

@stefanceriu stefanceriu left a comment

Choose a reason for hiding this comment

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

Not particularly familiar with the code but this looks sensible to me 👍

@Velin92 Velin92 merged commit 96940f6 into develop Jul 23, 2025
1 of 2 checks passed
@Velin92 Velin92 deleted the mauroromito/implement_new_MSC branch July 23, 2025 09:32
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 this pull request may close these issues.

3 participants