Skip to content

Comments

Update getUrlPreview to use /_matrix/client/v1/media/preview_url#5191

Open
Half-Shot wants to merge 4 commits intodevelopfrom
hs/update-media-prefix
Open

Update getUrlPreview to use /_matrix/client/v1/media/preview_url#5191
Half-Shot wants to merge 4 commits intodevelopfrom
hs/update-media-prefix

Conversation

@Half-Shot
Copy link
Contributor

This has been the case since v1.11 https://spec.matrix.org/v1.12/changelog/v1.11/

If we care about backwards compat further than v1.11 (so June 2024) then we can also do a switch for the older endpoint.

Checklist

  • Tests written for new code (and old code if feasible).
  • New or updated public/exported symbols have accurate TSDoc documentation.
  • Linter and other CI checks pass.
  • Sign-off given on the changes (see CONTRIBUTING.md).

@t3chguy
Copy link
Member

t3chguy commented Feb 16, 2026

If we care about backwards compat further than v1.11 (so June 2024) then we can also do a switch for the older endpoint.

We do, unless we want to remove <v1.11 support from https://github.com/matrix-org/matrix-js-sdk/blob/develop/src/version-support.ts#L27-L36 AIUI

@Half-Shot
Copy link
Contributor Author

oh well, it's like two lines.

Copy link
Member

@t3chguy t3chguy left a comment

Choose a reason for hiding this comment

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

This is not a T-Task, this should be in the changelog. Pick your poison of Defect/Enhancement

@Half-Shot Half-Shot added T-Defect T-Enhancement and removed T-Task Tasks for the team like planning T-Defect labels Feb 16, 2026
@Half-Shot
Copy link
Contributor Author

(I entirely forgot T-Tasks don't appear in the changelog. Going with Enhancement)

@t3chguy
Copy link
Member

t3chguy commented Feb 16, 2026

Would be good to have some tests, surprised this doesn't make Sonar sad tbh

image

@Half-Shot Half-Shot added this pull request to the merge queue Feb 17, 2026
github-merge-queue bot pushed a commit that referenced this pull request Feb 17, 2026
* Update preview_url endpoint.

* support <v1.11

* Add tests
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 17, 2026
@Half-Shot Half-Shot added this pull request to the merge queue Feb 19, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 19, 2026
Half-Shot added a commit to element-hq/element-web that referenced this pull request Feb 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants