Skip to content

Added biometric login for ios#120

Open
suleymanCel wants to merge 1 commit into
defaultfrom
112-ios-biometric-login
Open

Added biometric login for ios#120
suleymanCel wants to merge 1 commit into
defaultfrom
112-ios-biometric-login

Conversation

@suleymanCel

Copy link
Copy Markdown
Collaborator

No description provided.

@suleymanCel suleymanCel force-pushed the 112-ios-biometric-login branch from 78e2136 to 150c904 Compare February 13, 2023 16:19
Comment thread src/screens/BiometricSettings/BiometricSettings.tsx Outdated
Comment thread src/screens/BiometricSettings/BiometricSettings.tsx Outdated
Comment thread src/screens/BiometricSettings/BiometricSettings.tsx
Comment thread src/screens/BiometricSettings/BiometricSettings.tsx Outdated
Comment thread src/screens/BiometricSettings/BiometricSettings.tsx Outdated
(async () => {
const isBiometricSupport = await isBiometricSupported();
if (isBiometricSupport && isFirstSignIn) {
Alert.alert(

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Could we implement a custom modal with Telios styling not Alert?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I get the reference from our "Save Draft" alert.
-> But If you wish I can create a Modal as Telios styling and we can use where we need it instead of Alert.

Comment thread src/screens/LoginScreen.tsx Outdated
Comment thread src/util/asyncStorage.ts

@vitalikgriza vitalikgriza left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please check comments

@suleymanCel suleymanCel force-pushed the 112-ios-biometric-login branch from 14cee8d to 169bb20 Compare February 16, 2023 17:37
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.

2 participants