react-native-touch-id icon indicating copy to clipboard operation
react-native-touch-id copied to clipboard

Published 20 hours ago • verified publisher icon naoufal

Support Passcode Fallback along with Fingerprint

Open pawankhadpe opened this issue 5 years ago • 12 comments

Please review the changes and let me know if any changes needed.

pawankhadpe avatar Apr 10 '19 05:04 pawankhadpe

Hello,

can you review this merge request please. It is necessary for my current project to have a fallback if the fingerprint/face is not recognized :D

vdino96 avatar Jun 05 '19 09:06 vdino96

Hello,

What about merging this? I also need it for a work project ..

@pawankhadpe I may use your fork for this, would you mind explain me how to use the fallback?

Thanks!

aarknvir avatar Jun 13 '19 14:06 aarknvir

Why is this merge request pending for so long? please merge it if everything is working fine ? I am currently using the forked version by @pawankhadpe and its working as expected

divyanshunegi avatar Jun 29 '19 13:06 divyanshunegi

@naoufal @zibs this is absolutely critical functionality. The library is incomplete without it. It's a small change. Please review. Thanks.

fubar avatar Jul 08 '19 05:07 fubar

@naoufal @zibs I'll start sending reminders until we get someone's attention :p

fubar avatar Jul 10 '19 09:07 fubar

Because I couldn't wait for merge, I simply replaced this with LocalAuthentication from expo and it works. Maybe try that 😄

vdino96 avatar Jul 10 '19 09:07 vdino96

I've tried to avoid expo because of its extensive dependencies - but it quite likely is the safer bet if this library isn't maintained anymore.

fubar avatar Jul 10 '19 09:07 fubar

I highly recommend using Expo's LocalAuthentication since it is now available through their unimodules.

I don't have much time for maintaining this library right now, but would like to get into it better shape when I have time again, although I imagine even still the API would be nearly identical to theirs.

Thanks for the PR. I'll try to review it and merge + release it when I have the chance.

zibs avatar Jul 10 '19 16:07 zibs

Thanks @zibs!

fubar avatar Jul 10 '19 18:07 fubar

Any ETA on this, @zibs?

I understand you've other priorities as of now and may not get time to do this any time soon, but I'm debating on whether I should wait for this PR to be merged or spend time to switch to expo's unimodules. Would prefer the former if possible 😅

chirag64 avatar Aug 16 '19 15:08 chirag64

@aarknvir just passcodeFallback: true and it will work

@pawankhadpe thanks for your PR. Im using your repo :)

@zibs if you have the time, please have look this PR. Not everybody want to use Expo

siliconavengers avatar Sep 20 '19 08:09 siliconavengers

When will this be merged? This would be a great addition to the library.

KristineTrona avatar Nov 17 '20 11:11 KristineTrona