benjymous / MWM-for-Android

Meta Watch Manager Android SDK project for watch wrangling, remote protocol handling and notification routing for the Meta Watch wearable development systems. Includes an Intents based API for quick experiments.
www.metawatch.org
Apache License 2.0
59 stars 30 forks source link

Wunderground Weather without geo location #51

Closed dedee closed 11 years ago

dedee commented 11 years ago

In the current implementation comma char is being ignored

By default we use weather location "DALLAS,US"

This location is converted to this URL: http://api.wunderground.com/api/1df3eaacab9abc8a/geolookup/conditions/forecast10day/astronomy/q/DallasUS.json

Wunderground does not return valid data (also when entered in a browser).

When I am using the location string "DALLAS TX" it is working properly. http://api.wunderground.com/api/1df3eaacab9abc8a/geolookup/conditions/forecast10day/astronomy/q/Dallas%20TX.json

benjymous commented 11 years ago

Have changed it to replace commas with spaces, and changed the default location to "DALLAS TX" instead of "DALLAS,US"

benjymous commented 11 years ago

Fixed with above commit and 12bb389ae422aab1797cfd944d9b586367d611a2