|
|
1) Should 'Grant access to external storage' --> 'Grant access to storage' (i.e. doesn't this apply to internal and external storage)? 2) Re. 'Extended scanning', why is any configuration required? i.e. it's not reasonable to expect users to understand the mechanics of MM synchronization; shouldn't MMA automatically use 'extended sync metadata' automatically whenever it exists assuming that it is the most recent metadata? |
|
|
1) yes, since Android 11 this is related to internal storage too, so it can be changed to 'Grant access to storage' 2) it's possible when we take "Automatically" as the only option. |
|
|
2) As discussed offline, there aren't any cases where 'sync metadata' should be ignored. i.e. it should be used automatically / according to scan configuration and whatever case triggers a folder scan should also trigger an update of 'synced metadata' provided the synced metadata is more recent. Furthermore, whatever permissions are required to make it automatic should be requested (and that there should be a means of fixing the lack of such access if it hasn't been granted or was lost). Test note: this should work for: a) WiFi Sync b) USB Sync c) USB Sync to SD Card or OTG drive that's plugged into MMA d) Mixed content scenarios e.g 1 User DLs tracks to Android containing lots of metadata (which MMA doesn't see) 2 User Syncs some of those tracks to MMW which sees all of the metadata 3 MMW includes those tracks on the sync list to MMA --> User would expect to see the metadata from MMW in MMA Note: once MMA includes native tag support, we may want to rethink the logic slightly |
|
|
No Scan options screen in 1213. Added "Grant access to storage" button to "Library folders" only if some access permission is not granted. A user is asked to grant folder access on each mount storage event until he grants access or uses the "skip" button. V3 scan is launched automatically if fresh data are available and all needed permissions are granted. Fixed in build 2.0.1.1213 |
|
|
Tested 1213 and it seems to be working correctly. A couple of questions: i) If the user clicks the 'Skip' button on startup when folder access is requested, and they _ever_ prompted again to grant access? I'm asking because I would have expected that: - if MMW is configured with a Library folder that has not been granted access and MMW is configured to sync to that folder, that MMW would at least indicate on the Library Folders screen that there's an access problem, and provide a means of resolving it. Currently there's a vague message : Grant access to storage MediaMonkey requires write permission to some folders But: - It's not highlighted (i.e. it's easy to overlook) - it's vague in the sense that it doesn't highlight which of the folders are problematic ii) 'Grant access to storage' appears in Options > Library and Sync as well, but again, it just appears to be a regular configuration item, when in fact it's a warning. Perhaps the text should be highlighted in some manner and the string changed to 'MediaMonkey requires write permission to these folders to function optimally!' (this change works for the above case as well). |
|
|
i) The user is asked again before the first launch of wifi sync (or after the second step in the sync wizard). The folders in the Access dialog are updated by folders specified in MM5 and yes it would be nice to inform the user on the MM5 side. If the user uses the skip button then he is never asked again until some change happens(new storage, new setting in MM5). ii) A) highlight 1) At "Library folders" it's not a problem. We can change it to a warning color and we can also highlight folders. 2) At "Library and Sync" there is an issue. This is the preference screen so there are required only preference items. We can specify a custom layout for the option, but the general layout has slight differences depending on the Android version. I will verify the possibilities. B) text I believe the description should be shared between the option in "Library and Sync" and "Library scan", so there should not be "these folders", I suggest: MediaMonkey requires write permission to some folders to function optimally!' |
|
|
What do you think? I believe it's not too aggressive, warning enough though. Screenshot_20240920-154050.png (98,310 bytes) Screenshot_20240920-154104.png (211,067 bytes) |
|
|
Looks good, though I would change the text as follows: MediaMonkey requires write permissions for backup & USB sync. (no need to mention 'MediaMonkey folder' since it already appears in the Title) MediaMonkey requires folder write permissions to function optimally. (this is more concise and applies to both cases). |
|
|
Fixed in build 2.0.1.1214 |
|
|
Verified 1214 |
- Anonymous
