Skip to content
This repository has been archived by the owner on Oct 22, 2024. It is now read-only.

Use new CryptoEvent import #128

Conversation

florianduros
Copy link
Member

@florianduros florianduros commented Oct 7, 2024

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)

Task element-hq/element-web#26922
Following matrix-org/matrix-js-sdk#4443, replace the usage of the legacy crypto events by the events of the new crypto api.

…w-crypto-event-import

# Conflicts:
#	src/async-components/views/dialogs/security/CreateSecretStorageDialog.tsx
#	src/components/structures/MatrixChat.tsx
#	src/components/views/rooms/MemberTile.tsx
@richvdh richvdh self-requested a review October 16, 2024 10:11
Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

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

LGTM otherwise

playwright/e2e/crypto/utils.ts Outdated Show resolved Hide resolved
@florianduros florianduros added this pull request to the merge queue Oct 16, 2024
Merged via the queue into develop with commit 6684e79 Oct 16, 2024
27 checks passed
@florianduros florianduros deleted the florianduros/rip-out-legacy-crypto/use-new-crypto-event-import branch October 16, 2024 11:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Task Tasks for the team like planning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants