ubuntu-touch-coreapps-reviewers team mailing list archive
-
ubuntu-touch-coreapps-reviewers team
-
Mailing list archive
-
Message #00699
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