Skip to content

Conversation

@ovitrif
Copy link
Collaborator

@ovitrif ovitrif commented Apr 14, 2025

FIGMA Design

Biometrics & Pin on Launch Setup

Allows user to set PIN and toggle requiring it on launch. Also, on top of PIN, biometrics can be used.

  • Security & Privacy Settings screen with options:
    • Enable pin
    • Disable Pin logic (missing actual screen for it)
    • Require Pin on launch
    • Use biometrics instead
  • Require biometrics to unlock wallet on launch
    • Auth check UI with support for biometrics or pin
  • Security Sheet - flow screens
    • Prompt sheet (intro screen)
    • Choose PIN
    • Retype PIN + PIN incorrect
    • Ask For Biometrics screen + biometrics not available view
    • Result/ Confirm screen
  • Security logic to prevent onAuthenticationSucceeded hack scripting
  • Smooth opacity change on clickable elements

Previews

Flow to enable & unlock

flowEnablePinBio.mp4

Flow when biometrics are disabled on device

flowBioNotSupported.mp4

Still a lot of implementation needed to cover all cases and mirror the React Native functionality, but this PR is setting up a good base for next PRs to build on it with more granular progress.

ovitrif added 26 commits March 24, 2025 14:08
# Conflicts:
#	app/build.gradle.kts
#	gradle/libs.versions.toml
Currently only affects implementation for biometrics, but it helps wire up the logic and will be needed for actual pin logic as well.
Scripts can call `AuthenticationCallback.onAuthenticationSucceeded`, which is why we need to secure against such attacks.
@ovitrif ovitrif changed the title Biometrics setup Biometrics & Pin on Launch Setup Apr 18, 2025
@ovitrif ovitrif marked this pull request as ready for review April 18, 2025 14:50
@ovitrif ovitrif requested a review from jvsena42 April 18, 2025 14:50
@ovitrif ovitrif merged commit 39aacc2 into master Apr 21, 2025
1 check passed
@ovitrif ovitrif deleted the feat/biometrics-setup branch April 21, 2025 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants