AndyQ / NFCPassportReader

NFCPassportReader for iOS 13
MIT License
738 stars 239 forks source link

Introduced a new display message #172

Closed nmccann closed 11 months ago

nmccann commented 1 year ago

During testing integration with this library, we noticed that users were sometimes getting errors because they saw the "Passport read successfully" message (but not the checkmark) and then moved their phone away from the NFC chip, resulting in a read error.

It appears that this is due to the active authentication taking a bit of time to complete, and so it would seem useful to introduce messaging for this state before showing the "Passport read successfully" message.

An alternative approach would be:

AndyQ commented 1 year ago

Thanks - will add this in the next version

AndyQ commented 11 months ago

Merged - thanks