Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable cypress tests for shields with rust crypto #11589

Merged
merged 16 commits into from
Sep 18, 2023

Conversation

richvdh
Copy link
Member

@richvdh richvdh commented Sep 8, 2023

Fixes element-hq/element-web#25321
Based on #11528
Requires matrix-org/matrix-js-sdk#3718


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

@richvdh richvdh added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Sep 8, 2023
@richvdh richvdh marked this pull request as ready for review September 8, 2023 03:48
@richvdh richvdh requested a review from a team as a code owner September 8, 2023 03:48
@richvdh richvdh requested review from dbkr and kerryarchibald and removed request for a team September 8, 2023 03:48
Copy link
Member

@dbkr dbkr left a comment

Choose a reason for hiding this comment

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

lgtm although looks like they are still causing some failures?

Base automatically changed from rav/element-r/shields/new_api to develop September 18, 2023 17:45
@richvdh
Copy link
Member Author

richvdh commented Sep 18, 2023

lgtm although looks like they are still causing some failures?

I think this was in #11364, and got fixed.

Merged via the queue into develop with commit 50ee43c Sep 18, 2023
80 checks passed
@richvdh richvdh deleted the rav/element-r/shields/work branch September 18, 2023 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Element-R: Fix up message shields
3 participants