Export to public directory (Fix #443, Fix #440) #451
Conversation
|
Hi @Zahnstocher, I’ve reviewed the PR "Export to public directory (Fix #443, Fix #440)," and I can confirm it works perfectly in both the current version and with the updates in PR #457 (target SDK 35). Excellent work! However, I noticed a couple of points that could be improved: The pull request currently lacks a description. Including a brief overview of the changes and their impact would make the PR more informative for reviewers. The exported files are being saved in the Documents directory, which isn’t reflected in the current UI message. The message still mentions “External storage (SD) directory,” which might confuse users. Updating this text to accurately reflect the export location would improve clarity and usability. |
To improve clarity for users, you could update this message in the file app/src/main/res/values/strings-preferences.xml, specifically on line 76, to reflect the correct export location. If you’d like, I can take care of making this change for you—just let me know! |
|
Hi @Andyporras, about the string in file "strings-preferences.xml" line 76. I would set this string simply to "Public directory". Tell me if you're good with this or propose me something else 👍 |
Thank you for the suggestion! I think "Public directory (in documents)" would be a bit more descriptive and helpful for users, as it clarifies where the files will be stored. Let me know if that works for you or if you'd prefer keeping it simpler. This to make that change in pr #458. |
Changed the GPX export location to the Documents folder for better organization and user accessibility, as per feedback. Adjusted related strings and resolved conflicts with PR labexp#451
* SDK and MinSKD update * Change GPX export to user-accessible location Modifies the GPX export path to save files in a user-accessible location. Previously, files were exported to a default location that was not visible to the user, making access difficult. Exports are now saved in the Downloads folder. The solution works up to the latest version, Android 14. Co-Authored-By: Kevin Salazar <[email protected]> * Update strings-preferences.xml Changed the GPX export location to the Documents folder for better organization and user accessibility, as per feedback. Adjusted related strings and resolved conflicts with PR #451 * Update GPX export logic * Update GpsStatusRecord.java * Update About.java * Update ExportToStorageTask.java * Update strings-preferences.xml * Update ExportToStorageTask.java * Update ExportToStorageTask.java * Update AndroidManifest.xml --------- Co-authored-by: Kevin Salazar <[email protected]> Co-authored-by: Binnette <[email protected]>
No description provided.