Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Kill off references to deprecated getStoredDevice and getStoredDevicesForUser #11152

Merged
merged 9 commits into from
Jun 28, 2023

Conversation

richvdh
Copy link
Member

@richvdh richvdh commented Jun 27, 2023

These methods have been deprecated in the js-sdk, and don't work with rust crypto, so switch to their replacements.

Review commit-by-commit. Based on #11141.


This change is marked as an internal change (Task), so will not be included in the changelog.

richvdh and others added 3 commits June 26, 2023 16:20
matrix-org/matrix-js-sdk#3514 deprecates
`CryptoEvent.VerificationRequest` in favour of
`CryptoEvent.VerificationRequestReceived`. Use the new event.
@richvdh richvdh added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Jun 27, 2023
@richvdh richvdh force-pushed the rav/element-r/30_kill_get_stored_device branch from 43cc2b2 to e065f7e Compare June 27, 2023 20:25
@richvdh richvdh force-pushed the rav/element-r/30_kill_get_stored_device branch from e065f7e to 5621e30 Compare June 27, 2023 21:17
@richvdh richvdh marked this pull request as ready for review June 28, 2023 10:11
@richvdh richvdh requested a review from a team as a code owner June 28, 2023 10:11
@richvdh richvdh requested review from germain-gg and SimonBrandner and removed request for a team June 28, 2023 10:11
Base automatically changed from rav/element-r/29_verification_request_received_event to develop June 28, 2023 10:58
Copy link
Contributor

@germain-gg germain-gg left a comment

Choose a reason for hiding this comment

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

Looks sane 🚀

@richvdh richvdh added this pull request to the merge queue Jun 28, 2023
Merged via the queue into develop with commit 46eb34a Jun 28, 2023
@richvdh richvdh deleted the rav/element-r/30_kill_get_stored_device branch June 28, 2023 13:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants