Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Biometric Authentication to unlock the PwManager. Closes #28 #115

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Akshat1276
Copy link

Closes #28

Type of Change

  • New Feature
  • Bug Fix
  • Code Refactor
  • Documentation Update
  • Other (please specify):

Description of Change

Added Biometric Authentication to unlock the PwManager.

Implementation Details

enabled the authentication feature using WinBio (for windows machine) and objc(for apple machine). once authenticated successfully, the PwManager will start.

@IronJam11
Copy link
Collaborator

IronJam11 commented Jan 4, 2025

can you attach photos or videos to support your pr please?

@Akshat1276
Copy link
Author

@IronJam11 Sir I do not have a device with fingerprint sensor, so I do not have any demo photo or video, but I assure you that the code is correct.

@IronJam11
Copy link
Collaborator

Update the requirements.txt as need be as well so that it is easy to install the packages.

@Akshat1276
Copy link
Author

@IronJam11 I have made the advised changes.

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.

Integrate with biometrics
2 participants