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

Warn and block sending on verification violation #3679

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

BillCarsonFr
Copy link
Member

Pull Request Checklist

Draft as it depends on matrix-org/matrix-rust-sdk#4478
Part of element-hq/element-meta#2492 (still partial as there is no way to verify other users yet on EX, so you can only withdraw the verification requirement).
EXI counterpart of EXA element-hq/element-x-android#4126

Adds support for identity violation detection. Will show a warning notice and block sending (notice that sending would fail anyhow in that case).
image

withdraw_exi

UI changes have been tested with:

  • iPhone and iPad simulators in portrait and landscape orientations.
  • Dark mode enabled and disabled.
  • Various sizes of dynamic type.
  • Voiceover enabled.

Copy link

codecov bot commented Jan 15, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
911 1 910 0
View the top 1 failed tests by shortest run time
UserSessionFlowCoordinatorTests testEventRoutes()
Stack Traces | 1.57s run time
XCTAssertEqual failed: ("Optional(0)") is not equal to ("Optional(1)") (UnitTests/Sources/UserSessionFlowCoordinatorTests.swift:220)

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

@BillCarsonFr BillCarsonFr force-pushed the valere/identity_violation_banner branch from 1cc742b to 47671c0 Compare January 15, 2025 14:00
@BillCarsonFr BillCarsonFr added the pr-feature for a new feature label Jan 15, 2025
@BillCarsonFr BillCarsonFr changed the title feat(crypto): Warn and block sending on verification violation Warn and block sending on verification violation Jan 15, 2025
@BillCarsonFr BillCarsonFr force-pushed the valere/identity_violation_banner branch from e3422f9 to f7003cb Compare January 15, 2025 21:44
@BillCarsonFr BillCarsonFr force-pushed the valere/identity_violation_banner branch from 6b00504 to 0feecfa Compare January 16, 2025 13:18
@BillCarsonFr BillCarsonFr marked this pull request as ready for review January 16, 2025 13:18
@BillCarsonFr BillCarsonFr requested a review from a team as a code owner January 16, 2025 13:18
@BillCarsonFr BillCarsonFr requested review from Velin92 and removed request for a team January 16, 2025 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-feature for a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant