-
Notifications
You must be signed in to change notification settings - Fork 53
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
Modernize #19
base: master
Are you sure you want to change the base?
Modernize #19
Conversation
looks good to me, thanks 👍 I'll check it out on my machine later then merge |
@Valodim ? |
It's a pre work for modernize https://github.com/open-keychain/open-keychain |
Worked well for me but had to update
|
Surprisingly, someone takes care about this pull request !
|
Without this option, bindToService fails on newer Android API versions.
I was having issues running the example application as it wouldn't connect to OpenKeychain. It wouldn't see OpenKeychain. As soon as I added this option, I was able to select OpenKeychain as my encryption provider. I sent you a PR with the recommended change. |
Fix bindToService on newer Android API versions
Move the queries from the example application into the API
Please see commit messages for details