Skip to content

WIP: Added NFC lock feature#203

Draft
busycalibrating wants to merge 1 commit intonethical6:kt-rewritefrom
busycalibrating:feature/nfc-lock
Draft

WIP: Added NFC lock feature#203
busycalibrating wants to merge 1 commit intonethical6:kt-rewritefrom
busycalibrating:feature/nfc-lock

Conversation

@busycalibrating
Copy link

Implemented as another block set, toggled by an NFC tag which can be written by the app. Has an emergency unlock feature + an optional timer instead of only using the tag to unlock the apps.

Full disclosure: I have no idea what I'm doing with respect to Android development, and this was mostly Claude generated code. However I couldn't find a FOSS app blocker with support for NFC tags on android (though there were a few for apple devices), and since other people seem to want $100 AND a subscription for this, I figured I could just try adding this to digipaws. I've tested it on my phone (pixel 9) and it seems to work well, but if you don't want to deal with the bloat/potential bugs that this may introduce, feel free to close this PR! Maybe it could serve as inspiration for people who know what they're doing :)

implemented as another block set, toggled by an NFC tag which can be written by the app. Has an emergency unlock feature + an optional timer instead of only using the tag to unlock the apps.
@nethical6
Copy link
Owner

Hello!
I actually don't have a nfc supported device so there is no way for me to debug it.
Could you please test the app further for a few more weeks and confirm if it really works well.
Thanks!

@busycalibrating
Copy link
Author

Will do! So far it's been good, just have one more feature I'd like to add. Also I didn't notice #187 before doing this, feel free to merge in whichever PR you prefer if you choose to integrate this feature. Thanks for the app, it's great!

@busycalibrating
Copy link
Author

I've been using this for the last few weeks and it's been working well for me, would you be interested in merging it into the app or would you rather keep it separate?

@nethical6
Copy link
Owner

I've been using this for the last few weeks and it's been working well for me, would you be interested in merging it into the app or would you rather keep it separate?

thanks, i just wanted someone to test it.
I'll merge it soon.

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.

2 participants