Weather Indicator

Merge lp://qastaging/~jtasker/weather-indicator/cloudy into lp://qastaging/weather-indicator

  1. cloudy
  2. Merge into rainy
Proposed by Joshua Tasker
Status: Merged
Approved by: Vadim Rutkovsky
Approved revision: 312
Merged at revision: 312
Proposed branch: lp://qastaging/~jtasker/weather-indicator/cloudy
Merge into: lp://qastaging/weather-indicator
Diff against target: 174 lines (+36/-53)
1 file modified
bin/indicator-weather (+36/-53)
To merge this branch: bzr merge lp://qastaging/~jtasker/weather-indicator/cloudy
Reviewer Review Type Date Requested Status
Joshua Tasker Approve
Vadim Rutkovsky Approve
Review via email: mp+152550@code.qastaging.launchpad.net

Description of the change

Rewrote the Forecast code to pull from Yahoo Weather instead of the (now-discontinued) Google Weather API.
Requires an updated version of pywapi (the current Ubuntu package is out of date) which is available from here:
http://code.google.com/p/python-weather-api/source/browse/#svn%2Ftrunk

To post a comment you must log in.
Revision history for this message
Vadim Rutkovsky (roignac) wrote :

Looks good to me, however I can't verify the fix

review: Approve
Revision history for this message
Joshua Tasker (jtasker) wrote :

pywapi 0.3 should be "officially" posted later today, it uses revision r117 at the above google code link

review: Approve
Revision history for this message
Joshua Tasker (jtasker) wrote :

Just an FYI, pywapi 0.3 source is available here: http://code.google.com/p/python-weather-api/downloads/detail?name=pywapi-0.3.tar.gz
And debs from the PPA here: https://launchpad.net/~jtasker/+archive/pywapi/+packages

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: