-
Notifications
You must be signed in to change notification settings - Fork 105
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
Tech debt: Update method used to get recent calls list #1608
Comments
Also we should filter to rooms with known keys |
I could see that the following helpers for the room list could be very helpful:
The functions build up on each other and we could start with only the first one: This very much also makes sense to do server side at some point in time. Basically just cache and calculate: room.get_matrixRTC_sessions on the homeserver since otherwise each client would need to run the same algorithm for a room. This basically is the same as Daves initial post but sumarizes more concrete functions with the hope, that we can decide what exactly we need for the userexperience we aim for and how much effort the desired part would be while providing a long term plan to query matrixRTC related data. |
Your use case
We currently still fetch the recent calls list using the old GroupCall class. We should update it to something more MatrixRTC-y, although the semantics are room that has an m.call event (ie. that has the state event to say it has a call ongoing). This concept doesn't exist now, so the semantics will need to change slightly.
Ideas we have:
Since you can only join calls in the SPA anyway, and can't use your regular account, there's no practical difference as it stands currently. This would only affect things once we allowed federation / had people logging in with accounts used for things other than EC.
Have you considered any alternatives?
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: