Skip to content

Conversation

dbkr
Copy link
Member

@dbkr dbkr commented Aug 29, 2025

Update it to look for the stable-suffixed version too.

This will only be relevant until the next spec release but may as well fix it.

Requires matrix-org/matrix-js-sdk#4983

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.
  • I have licensed the changes to Element by completing the Contributor License Agreement (CLA)

It looked for the ".stable" suffixed feature to work out what URL
to use but not to see whether the server supported it.
@dbkr dbkr added the T-Defect label Aug 29, 2025
dbkr added a commit to matrix-org/matrix-js-sdk that referenced this pull request Aug 29, 2025
It looked for the ".stable" suffixed feature to work out what URL to use but not to see whether the server supported it.

This will only be relevant until the next spec release but may as well fix it.

See also element-hq/element-web#30649
dbkr added a commit to matrix-org/matrix-js-sdk that referenced this pull request Sep 1, 2025
* Fix stable-suffixed MSC4133 support

It looked for the ".stable" suffixed feature to work out what URL to use but not to see whether the server supported it.

This will only be relevant until the next spec release but may as well fix it.

See also element-hq/element-web#30649

* Fix awaiting
@dbkr dbkr added this pull request to the merge queue Sep 2, 2025
Merged via the queue into develop with commit 1b4a979 Sep 2, 2025
41 checks passed
@dbkr dbkr deleted the dbkr/fix_stable_msc4133_support branch September 2, 2025 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants