Update user permissions when room power levels change, or when they accept an invitation to a room

[aaravlu]

Fixes #302

[aaravlu]

Screencast.from.2024-12-27.19-56-49.webm

[kevinaboos]

Screencast.from.2024-12-27.19-56-49.webm

looks like things still don't work? i can't quite tell what this screen recording is trying to show.... it looks like you join a room but then it still shows that you don't have permissions to send a message?

it also looks like your changes broke the latest event code, since many of the rooms say [Loading latest message]

[kevinaboos]

Based on your screen recording, i think this PR accidentally breaks the latest_event calculation code.

Instead of this new approach where you separate out avatar handling from "can send message" handling, please move it all into a single update_latest_event() function (like i had before), which will ensure that all of the various possible relevant event kinds are handled upon every change to the latest event (which can occur in multiple places). I think the following 5 things are of interest to us:

• room name
• room avatar
• room power levels
• membership change
• latest event text preview

[aaravlu]

looks like things still don't work?

It works. I apologize for not clarifying the screenshot.

I have already used account aaravlu to set room 302 send permission to 100 so that account demolemon cannot send message in the room, and then aaravlu invited demolemon to this room.
It directly show the cannot send notice in demolemon's screen after accepting the invitation without reselecting room 302 manually.

[kevinaboos]

looks like things still don't work?

It works. I apologize for not clarifying the screenshot.

I have already used account aaravlu to set room 302 send permission to 100 so that account demolemon cannot send message in the room, and then aaravlu invited demolemon to this room. It directly show the cannot send notice in demolemon's screen after accepting the invitation without reselecting room 302 manually.

oh i see, that makes more sense then. It appeared to be the opposite. Thanks for clarifying

[kevinaboos]

Getting closer, but you missed some of what I said here:

Instead of this new approach where you separate out avatar handling from "can send message" handling, please move it all into a single update_latest_event() function (like i had before), which will ensure that all of the various possible relevant event kinds are handled upon every change to the latest event (which can occur in multiple places). I think the following 5 things are of interest to us:

• room name
• room avatar
• room power levels
• membership change
• latest event text preview

[aaravlu]

Getting closer, but you missed some

Yeah. now they are in one function.
We currently handle room name, avatar, membership and power levels changes.
Lmk if there's anything else you want.

[kevinaboos]

Looks good now! Thanks for your great work here!