woheller69 / omweather

GNU General Public License v3.0
401 stars 17 forks source link

Explanation for "update interval" option #101

Open Cwpute opened 1 month ago

Cwpute commented 1 month ago

Problem The update interval is only for when the app is already open, or for the widget if you have it set up. This is not clearly explained in the options and one can assume the app will update in the background.

Suggested solution Provide a little explanation text under that option to say the app.never updates in the background and that this is for the widge only.

Environment (please complete the following information):

woheller69 commented 1 month ago

This is not the widget update intervall.

Intervall 1h means that the app will update data if you open that tab again after more than 1h since last update

Cwpute commented 1 month ago

Understood. I was mislead by this comment of yours: https://github.com/woheller69/omweather/issues/81#issuecomment-2134492770 I still think the option would benefit from this little explanation, as providing a time interval makes one think this is independent of any factor, like having the app open, or having fetched weather data less time ago than the set time interval.

woheller69 commented 1 month ago

updating in background would not have any effect if you do not look at it. The widget (and with it the first tab) will update in background or when you unlock device. This is controlled by Android mechanisms. If you allow GPS for the widget it will also check for location changes about every 10 min and update in case of a major change.

Cwpute commented 1 month ago

Yes. Can you include a paragraph explaining that behavior under the time interval option ?

woheller69 commented 1 month ago

I will rather remove that option as it mainly confuses people and does not make a lot of sense. Open-Meteo weather models are updated once per hour so I might simply set a fixed value of 30 min

Cwpute commented 1 month ago

Oh yeah, definitely ! if models aren't updated that often, as far as i'm concerned, this is very much an acceptable solution 👍