Improve OpenStreetMap edits upload #7402
Labels
No labels
Accessibility
Accessibility
Address
Address
Android
Android
Android Auto
Android Auto
Android Automotive (AAOS)
Android Automotive (AAOS)
API
API
AppGallery
AppGallery
AppStore
AppStore
Battery and Performance
Battery and Performance
Blocker
Blocker
Bookmarks and Tracks
Bookmarks and Tracks
Borders
Borders
Bug
Bug
Build
Build
CarPlay
CarPlay
Classificator
Classificator
Community
Community
Core
Core
CrashReports
CrashReports
Cycling
Cycling
Desktop
Desktop
DevEx
DevEx
DevOps
DevOps
dev_sandbox
dev_sandbox
Directions
Directions
Documentation
Documentation
Downloader
Downloader
Drape
Drape
Driving
Driving
Duplicate
Duplicate
Editor
Editor
Elevation
Elevation
Enhancement
Enhancement
Epic
Epic
External Map Datasets
External Map Datasets
F-Droid
F-Droid
Fonts
Fonts
Frequently User Reported
Frequently User Reported
Fund
Fund
Generator
Generator
Good first issue
Good first issue
Google Play
Google Play
GPS
GPS
GSoC
GSoC
iCloud
iCloud
Icons
Icons
iOS
iOS
Legal
Legal
Linux Desktop
Linux Desktop
Linux packaging
Linux packaging
Linux Phone
Linux Phone
Mac OS
Mac OS
Map Data
Map Data
Metro
Metro
Navigation
Navigation
Need Feedback
Need Feedback
Night Mode
Night Mode
NLnet 2024-06-281
NLnet 2024-06-281
No Feature Parity
No Feature Parity
Opening Hours
Opening Hours
Outdoors
Outdoors
POI Info
POI Info
Privacy
Privacy
Public Transport
Public Transport
Raw Idea
Raw Idea
Refactoring
Refactoring
Regional
Regional
Regression
Regression
Releases
Releases
RoboTest
RoboTest
Route Planning
Route Planning
Routing
Routing
Ruler
Ruler
Search
Search
Security
Security
Styles
Styles
Tests
Tests
Track Recording
Track Recording
Translations
Translations
TTS
TTS
UI
UI
UX
UX
Walk Navigation
Walk Navigation
Watches
Watches
Web
Web
Wikipedia
Wikipedia
Windows
Windows
Won't fix
Won't fix
World Map
World Map
No milestone
No project
No assignees
1 participant
Due date
No due date set.
Dependencies
No dependencies set.
Reference: organicmaps/organicmaps-tmp#7402
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
The issue is more critical for Android, but iOS implementation will also benefit from it.
ATM OM uploads edits when it's moved into the background/switched to another app.
The main Android issue is that the system may kill or suspend the app and break the upload, which leads to changesets without comments [TODO: insert existing issues here] or failed uploads [list issues here]
The important UX issue is that users don't understand when/how their edits are uploaded.
Another issue is about privacy [reference it here]. Although single edits are not uploaded immediately, but later, in batches, a bigger delay may be needed to dissociate a person from recently visited places.
Here are some ideas on how to implement it:
Random delay is not needed for many users. Many OSMers and non-geeky users would love to see their changes uploaded immediately and will complain about any delay.
Random upload delay is needed for those who care about privacy.
Any upload delay is needed to batch changes and don't DDOS OpenStreetMap.org servers (yes, we are thinking about the future now, with millions of active OM users worldwide). Remember that OpenStreetMap.org is not a powerful corporation with many resources to scale and protect servers.
Showing (and then hiding on successful upload) a foreground service notification while the app is active is annoying.
For most users, the default master branch batching approach should be enough, when changes are uploaded when OM goes into the background. Meaning that a notification appearing when OM goes into the background would be helpful to see that everything was uploaded smoothly or failed.
On success, a notification may report success for a second and disappear. On failure, it may stay there and handle different errors differently.
It would be great to show more detail about failed edits on tap.
For privacy mode an (off by default) checkbox may be introduced in settings (should we make a Privacy section there?):
"Delays OSM edits upload for a random amount of time".
A timer can be displayed in the notification in this case, with "tap to upload immediately" functionality.
Originally posted by @biodranik in organicmaps/organicmaps#6119 (comment)