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

Declare package visibility needs. #108

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

Conversation

guilhermeagostinelli
Copy link
Contributor

@guilhermeagostinelli guilhermeagostinelli commented Jun 15, 2022

According to Google:

As you create your app, it's important to consider the set of other installed apps on the device that your app intends to access. If your app targets Android 11 (API level 30) or higher, the system makes some apps visible to your app automatically, but it filters out other apps by default.

If your app targets Android 11 or higher and needs to interact with apps other than the ones that are visible automatically, add the <queries> element in your app's manifest file.

This PR addresses this issue by specifying which packages to query in the AndroidManifest file, so the checkInstalledAppsForShare function can work properly.

Fixes #81

@guilhermeagostinelli
Copy link
Contributor Author

p.s. another option would be to specify in the README the need of adding the <queries> element to the app's AndroidManifest.xml, thus transferring the responsibility of this step to the user. This would allow the user to specifically choose which packages he would like to add under the <queries> tag.

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.

everything is false in checkInstalledAppsForShare
1 participant