← Back to team overview

ubuntu-touch-coreapps-reviewers team mailing list archive

Re: [Merge] lp:~vthompson/ubuntu-weather-app/reboot-get-initial-data-from-storage into lp:ubuntu-weather-app/reboot

 

Ok, one last thought on this. I tried your MP with the solution below and it works well. Could you propose your reboot-worker mp with the following change?

12	-    Component.onCompleted: refreshData();
13	+    Component.onCompleted: {
14	+        storage.getLocations(fillPages);
15	+        refreshData();
16	+    }

I think always starting with stored data and then spinning off the request in a worker is probably the best solution for now. What do you think?
-- 
https://code.launchpad.net/~vthompson/ubuntu-weather-app/reboot-get-initial-data-from-storage/+merge/252962
Your team Ubuntu Weather Developers is requested to review the proposed merge of lp:~vthompson/ubuntu-weather-app/reboot-get-initial-data-from-storage into lp:ubuntu-weather-app/reboot.


References