Merge lp:~martin-borho/ubuntu-weather-app/backend-upgrade into lp:ubuntu-weather-app/obsolete.trunk
Proposed by
Martin Borho
Status: | Merged |
---|---|
Approved by: | Raúl Yeguas |
Approved revision: | 40 |
Merged at revision: | 41 |
Proposed branch: | lp:~martin-borho/ubuntu-weather-app/backend-upgrade |
Merge into: | lp:ubuntu-weather-app/obsolete.trunk |
Diff against target: |
275 lines (+93/-43) 5 files modified
components/LocationTab.qml (+8/-7) components/SettingsDialog.qml (+1/-1) components/Storage.qml (+1/-0) components/WeatherApi.js (+66/-26) ubuntu-weather-app.qml (+17/-9) |
To merge this branch: | bzr merge lp:~martin-borho/ubuntu-weather-app/backend-upgrade |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Raúl Yeguas | Approve | ||
Ubuntu Phone Apps Jenkins Bot | continuous-integration | Approve | |
Review via email: mp+169423@code.launchpad.net |
Commit message
Improvements to API-Worker to reduce HTTP calls against weather api.
Description of the change
- Weather data will not get refreshed unless stored data is older than 30min or an refresh is forced.
- Metric and imperial values are getting precalculated in API-Worker now. Refresh of weather data isn't neccessary anymore after changing scale in settings.
To post a comment you must log in.
PASSED: Continuous integration, rev:40 91.189. 93.125: 8080/job/ ubuntu- weather- app-ci/ 20/ 91.189. 93.125: 8080/job/ ubuntu- weather- app-quantal- amd64-ci/ 17 91.189. 93.125: 8080/job/ ubuntu- weather- app-raring- amd64-ci/ 20
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild: 91.189. 93.125: 8080/job/ ubuntu- weather- app-ci/ 20/rebuild
http://