Voicerra icon indicating copy to clipboard operation
Voicerra copied to clipboard

implement app lock for notes section of the app

Open TheGuyDangerous opened this issue 2 years ago • 9 comments

TheGuyDangerous avatar Oct 27 '22 03:10 TheGuyDangerous

Hey @TheGuyDangerous Can you please explain what has to be done in detail? Thanks

chandansgowda avatar Nov 12 '22 03:11 chandansgowda

Sure !!☺️

TheGuyDangerous avatar Nov 12 '22 08:11 TheGuyDangerous

For now , the about_page.dart doesn't contain any settings content .. suitable to its name -

image

So i'm planning to implement some settings content to the page, over time. One of the settings being an App lock to the notes page -

Here's a sample render of what some of the settings would look like - So, I'd need a new Option here -

with the title 'App lock' and a dropdown to enable biometric You can use the local_auth package for implementing local , on-device authentication of the user.(if possible ,only on notes page if not , we'll discuss)

TheGuyDangerous avatar Nov 12 '22 08:11 TheGuyDangerous

Thanks @TheGuyDangerous Assign this to me if its priority is low. Coz I'll be working on it after Wednesday. If it has to be done ASAP, then its okayy.

chandansgowda avatar Nov 12 '22 12:11 chandansgowda

Thanks @TheGuyDangerous Assign this to me if its priority is low. Coz I'll be working on it after Wednesday. If it has to be done ASAP, then its okayy.

Take all the time you need ☺️ @chandansgowda !! I've assigned it to you

TheGuyDangerous avatar Nov 12 '22 13:11 TheGuyDangerous

Hey @TheGuyDangerous I have setup the UI as required. I have also used shared preferences and local_auth to setup app lock using biometrics. But I'm not able to ask for authentication when a user goes into the notes section. Shall I raise a pull request with what I have done? We can raise a separate issue to add it to notes page later by continuing this issue.

chandansgowda avatar Dec 02 '22 10:12 chandansgowda

umm.. yeah .. I looked at it .. you can send in the pr.. and try to implement it in the meantime :)

Just make sure to push the pr to the dev branch

TheGuyDangerous avatar Dec 02 '22 21:12 TheGuyDangerous

Hey @TheGuyDangerous I have raised a PR - #23 Please review.

chandansgowda avatar Dec 03 '22 02:12 chandansgowda

awesome @chandansgowda

TheGuyDangerous avatar Dec 03 '22 07:12 TheGuyDangerous