Code review comment for lp:~nik90/ubuntu-clock-app/sync-clock-backend

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

@dpm, thnx for the info. I will be testing the qtlocation this weekend. While using qtlocation, i had a few crashes on the desktop which is not good. Anyways this branch is more focussed on fixing the default time shown by the clock app and ensure it is the same as the system time. As a bonus, I added the geoIP feature for automatic location retrieval. So from the perspective I consider this MP complete and ready for trunk merge.

As said above, I will test qtlocation in the weekend and propose that in a new branch by itself to spot any regression due to qtlocation quicker. Please approve if you agree.

« Back to merge proposal