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

[BUG] Images fail to download to SD card if set as default download directory #435

Open
geominorai opened this issue Aug 23, 2022 · 6 comments
Assignees
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@geominorai
Copy link

Describe the bug

If the download path is set to an external SD card, the media file will attempt to download but fail without displaying any errors.

This does not happen when set to internal storage and media downloads to SD card succeed if download handing is set to always ask instead.

This bug did not exist prior to v3.0.0-beta1.

To Reproduce

Steps to reproduce the behavior:

  1. Set a folder on an SD card for download handling
  2. Open a Twitter image post
  3. Attempt to download

Device

Model: Samsung Galaxy S9+
OS: Android 10
Language: English
Fritter Version: v3.0.0-beta1

@geominorai geominorai added the bug Something isn't working label Aug 23, 2022
@jonjomckay
Copy link
Owner

This one might be difficult for me to fix, as I don't own a device with an SD card slot (I wish I did!).

I can add some more logging around download failures, but you might need to send me a logcat trace. I'll post back here when that's the case though.

@jonjomckay jonjomckay added the help wanted Extra attention is needed label Aug 30, 2022
@jonjomckay jonjomckay modified the milestone: v3.0.0 Aug 30, 2022
@jonjomckay jonjomckay self-assigned this Aug 30, 2022
@johann-gambol
Copy link
Contributor

johann-gambol commented Sep 16, 2022

I can't reproduce this bug with the current version. It's working for me with commit fb4b1b2 and with 3.0.0-beta 3.

Model: Samsung Galaxy S5
OS: Android 10
Language: German

@geominorai
Copy link
Author

I updated to that CI build but it still happens:

Fritter.mp4

@jonjomckay
Copy link
Owner

@geominorai can you try reselecting the directory on the Settings screen, and trying again?

@geominorai
Copy link
Author

geominorai commented Sep 23, 2022

I just updated to the CI for 9acbab6 and set the download directory again, but to no avail.

@geominorai
Copy link
Author

geominorai commented Sep 23, 2022

This is the error being shown with the latest CI:

This happens even if the download folder gets created in the directory selection popup in settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants