mapsme / omim

🗺️ MAPS.ME — Offline OpenStreetMap maps for iOS and Android
Apache License 2.0
4.57k stars 1.15k forks source link

Android background service #455

Open vojtechkral opened 8 years ago

vojtechkral commented 8 years ago

I noticed there's a Maps.me background service that starts on boot. What does this service do? Could it be made configurable?

biodranik commented 8 years ago

Now it is not yet configurable. It checks if maps are downloaded for region where you are now, and recommends to download them. Also it checks if map updates are available.

vojtechkral commented 8 years ago

Okay, thanks for info. Would you be open to the idea of making this configurable?

biodranik commented 8 years ago

Yes, it is a good idea and needs to be discussed. In general, we try to avoid many different settings and satisfy 99% of users automatically. OSMand is an opposite example which we don’t want to follow to stay simple.

May be it can be easier to incorporate it in some “hidden” advanced settings.

RicoElectrico commented 8 years ago

Yeah, but consider that many view unsolicited notifications as a nuisance. As per Google design guidelines, notifications should be made optional.. Do I have to mention that fail when you pushed "car free day" notifications in every single language? screenshot_2015-09-21-16-45-32 As per OsmAnd, it's not even about its functionality, but about its clunky interface. If you want to distance yourself from it, at least know why. Google/Apple Maps have very sophisticated features, but they're behind a simple and easy-to-use interface.

vojtechkral commented 8 years ago

Well, the setting could probably be described in simple terms. Such as, for example, "Check for map updates automatically" or something along those lines...

biodranik commented 8 years ago

Push notifications bug is not related at all to discussed settings (and discussed setting would not stop it even if it existed).

biodranik commented 8 years ago

We'll discuss it with our team, but it's definitely not a top priority for us right now. You can, however, add it in your own Pull Request :)

vojtechkral commented 8 years ago

Okay :)