Skip to content

client-api: add unstable MSC3266 support (room summary)#1776

Merged
bnjbvr merged 6 commits intoruma:mainfrom
bnjbvr:msc3266-room-summary
Apr 21, 2024
Merged

client-api: add unstable MSC3266 support (room summary)#1776
bnjbvr merged 6 commits intoruma:mainfrom
bnjbvr:msc3266-room-summary

Conversation

@bnjbvr
Copy link
Contributor

@bnjbvr bnjbvr commented Apr 16, 2024

This implements experimental support for MSC3266 for getting a room summary, which is really nice to get room previews even for rooms one is not part of.


Preview Removed

Copy link
Contributor

@Kladki Kladki left a comment

Choose a reason for hiding this comment

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

You also need to add an unstable feature for this in ruma and ruma-client-api

@bnjbvr bnjbvr force-pushed the msc3266-room-summary branch from ede426e to c6d5a2c Compare April 16, 2024 18:16
@bnjbvr bnjbvr requested a review from Kladki April 18, 2024 09:42
@bnjbvr bnjbvr force-pushed the msc3266-room-summary branch from 8e8f531 to b15fa72 Compare April 18, 2024 09:44
@bnjbvr bnjbvr requested review from Hywan and poljar and removed request for Kladki April 18, 2024 09:45
@bnjbvr bnjbvr force-pushed the msc3266-room-summary branch from b15fa72 to 7334c2c Compare April 18, 2024 09:47
@bnjbvr bnjbvr enabled auto-merge (squash) April 18, 2024 09:49
Co-authored-by: Matthias Ahouansou <matthias@ahouansou.cz>
Copy link
Contributor

@zecakeh zecakeh left a comment

Choose a reason for hiding this comment

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

In the MSC there are also changes to the hierarchy endpoints, could you add those for completeness?

bnjbvr and others added 2 commits April 19, 2024 16:53
Co-authored-by: Kévin Commaille <76261501+zecakeh@users.noreply.github.com>
Co-authored-by: Kévin Commaille <76261501+zecakeh@users.noreply.github.com>
@bnjbvr
Copy link
Contributor Author

bnjbvr commented Apr 19, 2024

In the MSC there are also changes to the hierarchy endpoints, could you add those for completeness?

Not sure what you're talking about, and please let's not scope-creep by requesting more work; there's been already enough time spent on this PR :-)

@bnjbvr bnjbvr merged commit cc44459 into ruma:main Apr 21, 2024
@bnjbvr bnjbvr deleted the msc3266-room-summary branch April 22, 2024 08:23
@bnjbvr
Copy link
Contributor Author

bnjbvr commented Apr 22, 2024

Thanks @zecakeh for the review ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants