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

[REQ] Enable access to exported settings on Android #723

Closed
grantland opened this issue Mar 9, 2022 · 22 comments
Closed

[REQ] Enable access to exported settings on Android #723

grantland opened this issue Mar 9, 2022 · 22 comments
Assignees
Labels
enhancement New feature or request pinned

Comments

@grantland
Copy link
Contributor

Is your feature request related to a problem? Please describe.
On some Android devices we're unable to access the directory and files of settings that are exported. This seems to be due to Scoped Storage on newer Android versions.

Describe the solution you'd like
Support sharing settings from within QZ by long pressing a settings export using the OS share intent system.

Describe alternatives you've considered
Adding Storage permissions, but it seems like they might have been removed in the past due to issues: https://developer.android.com/about/versions/11/privacy/storage

Additional context
Add any other context or screenshots about the feature request here.

@grantland grantland added the enhancement New feature or request label Mar 9, 2022
@cagnulein
Copy link
Owner

@p3g4asus could you help on this?

@p3g4asus
Copy link
Contributor

p3g4asus commented Mar 9, 2022

@grantland You should be able to access QZ folders on Android by installing those two free apps:
Google Files
and
File Manager
Then use File Manager to access /storage/emulated/0/Android/data/org.cagnulen.qdomyoszwift/files.
The first time you try to enter Android/data folder from File Manager it will open Google Files where you will be asked to grant access to that folder.
Please tell me if this workaround helps.
As for the request: I will try to study the way to do and to implement it this but it will not be immediate because accessing Android API from QT is a little bit cumbersome.

@grantland
Copy link
Contributor Author

On Android 11+ /storage/emulated/0/Android/data is inaccessible by file management apps: https://developer.android.com/about/versions/11/privacy/storage#file-access

@p3g4asus
Copy link
Contributor

I have pixel 4 XL not rooted, with Android 12 and stock rom and I can access that folder as I described in the above post.

@cagnulein
Copy link
Owner

@grantland maybe due to a custom rom flavour?

@grantland
Copy link
Contributor Author

Hrm weird as I'm using a non-rooted Pixel 4 with Android 12 as well.

It's strange to me that you're able to access it given that the docs also state that it should be inaccessible and my devices corroborates it.

@grantland
Copy link
Contributor Author

Did a quick search and other people experience this as well with any app downloaded from Google Play such as Google Files: https://www.reddit.com/r/AndroidPreviews/comments/iqcpq0/can_yall_access_the_androiddata_folder_it_shows/?utm_medium=android_app&utm_source=share

It seems like the only way to access that dir is via the hidden system Files app which you can only access from Settings > Storage > Files.

I tested that and it works, but it's very hidden.

@cagnulein
Copy link
Owner

at least there is a workaround :) thanks for reporting this!

@p3g4asus
Copy link
Contributor

I implemented this request. Please look at this video.

@grantland Please test it: You can find it here

@cagnulein
Copy link
Owner

great! thanks @p3g4asus ! let me know @grantland if it's ok to
you too!

@cagnulein
Copy link
Owner

@grantland any news?

@stale
Copy link

stale bot commented Apr 9, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Apr 9, 2022
@cagnulein
Copy link
Owner

@grantland Are you ok? I'm starting to worrying about you

@stale stale bot closed this as completed Apr 19, 2022
@cagnulein cagnulein reopened this Apr 19, 2022
@stale stale bot removed the wontfix This will not be worked on label Apr 19, 2022
@grantland
Copy link
Contributor Author

It works, thanks @p3g4asus!

@cagnulein
Copy link
Owner

@p3g4asus can you do a PR? Thanks

@p3g4asus
Copy link
Contributor

@p3g4asus can you do a PR? Thanks

Sure I will. But can I take some more time to try to fix the "Other folders" load settings issue? I will try to fix it and, if it reveals to be to cumbersome, I will remove the Other folders button and make a PR.

@cagnulein
Copy link
Owner

@p3g4asus yes Hi i totally agree! I guess I will merge this into the 2.11 series

@stale
Copy link

stale bot commented May 5, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label May 5, 2022
@cagnulein
Copy link
Owner

@p3g4asus are you waiting @grantland for this?

@stale stale bot removed the wontfix This will not be worked on label May 5, 2022
@p3g4asus
Copy link
Contributor

p3g4asus commented May 5, 2022

@p3g4asus are you waiting @grantland for this?

Yes I am. I am ready for PR.

@stale
Copy link

stale bot commented May 20, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label May 20, 2022
@cagnulein cagnulein added pinned and removed wontfix This will not be worked on labels May 21, 2022
@cagnulein
Copy link
Owner

this should be fixed by the #1868

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pinned
Projects
None yet
Development

No branches or pull requests

3 participants