-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Fix trigger media scan #2613
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
Fix trigger media scan #2613
Conversation
0c4ece0
to
c2e1213
Compare
c2e1213
to
7d3ba83
Compare
@jesmrec media scan triggering stopped working due to some changes we performed to make uploads and downloads with wrapped apps properly worked so could you test them to see that are still working? |
@jesmrec code approved, ready to test |
(1)I think this is not a problem in the oC app, but just in case: .mov files are not displayed in the Gallery (after checking different formats). Am i wrong or can we do something? @davigonz |
(2) READY TO TEST
about this. I compared the behaviour in version 2.11.0 and the current fix. With the fix, the pics are not displayed after being downloaded. Unfortunately, the behaviour is not the same (due to the fix?) MI wrapper 8.5 Gen1 With Android 2.11.0, pics downloaded and displayed correctly: With the fix, download error Tested with Android 5X v8 and Samsung S9 v9 I can provide as many wrapped apks as needed. |
.mov is a format created by Apple so I guess Android does not support it by default. Some users reported the same problem on S9 devices => https://www.reddit.com/r/GalaxyS9/comments/9t9dif/mov_files_not_showing_in_my_gallery/ |
(2) is fixed. But here we have gone to the scratch point. Now, pics are correctly downloaded and previewed in both wrapped and non-wrapped apps. But the Samsung gallery only shows the pics when the app is non-wrapped. That means, the initial bug persists if the app is MI-wrapped. Any other workaround is posible here @davigonz? |
This is how the wrapped apps work, files contained in the wrapped app container should not be accessible from other apps (including the gallery), but only from other wrapped apps. Otherwise, the wrapped apps would not be secure since they would be exposing their data to any app. |
Ok, then, everything fixed. Approved on my side. |
Fixes #2612
QA
Test plan: https://github.com/owncloud/QA/blob/master/Mobile/Android/Release_2.12.0/2613%20-%20trigger%20media%20scan.md
Bugs & improvements: