Closed t3chguy closed 2 years ago
Going to hold off landing this until Synapse has support for it, as it does add an additional API hit so a tiny bit of delay to HSes without support for this API - it is negligible but we gain nothing from landing it early
The Synapse MSC is getting close and I'm going on holiday, merging now, no rush on deploy, will create a card re deploying closer to go live
FYI the required APIs are now live on matrix.org, it sounds like we'll want to wait for it to be in a release version before deploying though since (I think) matrix.to hits various homeservers?
Yes, the first time you use matrix.to we ask you which homeserver to use to show previews..
(to be honest, this doesn't seem worth the likely drop-off just to show summaries ..)
the idea is that the "use matrix.org?" dialog should feel no more or less invasive than a GDPR click-thru fwiw. previews are pretty important UX though - in terms of room name, avatar, etc, otherwise it could just come up with a raw ID which is horrific.
Fixes https://github.com/matrix-org/matrix.to/issues/213 Fixes https://github.com/matrix-org/matrix.to/issues/214 Implements https://github.com/matrix-org/matrix-doc/pull/3266