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 MANAGE_EXTERNAL_STORE to manifest. This way gwc-files can be ac… #376

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

Conversation

rascheC
Copy link

@rascheC rascheC commented Jan 24, 2022

Added MANAGE_EXTERNAL_STORE to manifest. This way gwc-files can be accessed when on API 30 or higher. The permission needs to be set manually on my S10 under settings. Caution: According to the Google Playstore policy that means that the app will be further evaluated by Google.

…cessed when on API 30 or higher. The permission needs to be set manually on my S10 under settings.

Caution: According to the Google Playstore policy that means that the app will be further evaluated by Google.
@cgeo-ci-bot
Copy link
Collaborator

Can one of the core team members give me the the permission to test this?

@bekuno
Copy link
Member

bekuno commented Jan 24, 2022

ok to test

@SammysHP
Copy link
Member

SammysHP commented Jan 24, 2022

Sadly this is not a viable solution. Google won't allow this (only file managers and some other special apps might request this permission), on the other hand the implementation between different ROMs can vary. If things would be so easy we would have done it already.

@bekuno
Copy link
Member

bekuno commented Jan 24, 2022

@bekuno bekuno added the Do not merge / WIP Not for merging and/or work still in progress label Jan 24, 2022
@Lineflyer
Copy link
Member

We need to have scoped storage implemented... This change is no option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do not merge / WIP Not for merging and/or work still in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants