Skip to content

Conversation

@DorianMazur
Copy link
Collaborator

No description provided.

@DorianMazur DorianMazur merged commit 7eaf30e into master Oct 24, 2024
@DorianMazur DorianMazur deleted the release-9.1.0 branch October 24, 2024 09:51
* @platform iOS, visionOS
* @default AUTHENTICATION_TYPE.DEVICE_PASSCODE_OR_BIOMETRICS
*/
authenticationType?: AUTHENTICATION_TYPE;
Copy link

@joarkosberg joarkosberg Nov 11, 2024

Choose a reason for hiding this comment

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

A question, I see authenticationType parameter is not part of SetOptions or GetOptions. Starting seeing type error after updating to 9.1.0 and updating types.
Should it still be in those types or does the option dont provide anything for set/get password?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@joarkosberg Yes, that's right. The types and methods in this project were quite messy, so I tried to tidy them up a bit. The authenticationType is only needed and used in the canCheckAuthentication method.

Choose a reason for hiding this comment

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

Thanks for answer 😊

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