[TIMOB-25270] iPhone X: Support for FaceID
| GitHub Issue | n/a |
|---|---|
| Type | New Feature |
| Priority | High |
| Status | Closed |
| Resolution | Fixed |
| Resolution Date | 2017-09-18T08:12:12.000+0000 |
| Affected Version/s | n/a |
| Fix Version/s | Release 6.3.0 |
| Components | iOS |
| Labels | faceid, ios11, iphonex |
| Reporter | Hans Knöchel |
| Assignee | Hans Knöchel |
| Created | 2017-09-13T07:23:57.000+0000 |
| Updated | 2017-11-01T22:31:17.000+0000 |
Description
As discussed on Teams, we may want to create a new module like "Ti.Identity" to manage all security-related functionalities in one module, including touch-id, face-id and keychain-access. This will also deprecate Ti.TouchID and move Android over as well.
Ok, here we go: I've created a new module "Ti.Identity" and moved everything over from "Ti.TouchID". We would also need to deprecate Ti.TouchID now (e.g. a warning in the TouchId.yml file). Then we would need [~bimmel] to update his build scripts to add the following repo to his module repo list: https://github.com/appcelerator-modules/titanium-identity The module also represents the Face ID relevant changes: * New constants to see which biometrics are supported * New error-codes to handle Face ID errors Rather than that, Face ID will work together with Touch ID as the "biometrics" option, so technically we already support it with the current Ti.TouchID. But to keep it more generic for the future and semantic naming, this should be the right step to go.
[~hknoechel] I have reviewed the code of module. It looks good to me. There is one minor doc update required, which I had send you in chat. Once that change is done, my review for module is complete.
Validated new feature on both Android and iOS. Node Version: 6.10.5 NPM Version: 3.10.10 Mac OS: 10.13 Appc CLI: 6.3.0-master.16 Appc CLI NPM: 4.2.10-2 Titanium SDK version: 6.3.0.v20171031181249 Appcelerator Studio vers 4.10.0 Android 7.1.2 (Pixel) Xcode 9.1/iOS 11.0.3 iPhone 7 Plus