[ 
https://issues.apache.org/jira/browse/CAMEL-6398?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13667924#comment-13667924
 ] 

Babak Vahdat commented on CAMEL-6398:
-------------------------------------

[Polished|https://git-wip-us.apache.org/repos/asf?p=camel.git;a=commitdiff;h=eb1fa829;hp=1c7ed3d5f01f080bfaaf0898b2769f6d1322b871]
 the code base with the following changes:

- Replaced the boilerplate codes for reading the HTTP payloads with the Camel 
TypeConverter facility which would also close the streams automatically for 
free (see {{IOConverter#toString(url, exchange)}})
- Check for proper HTTP replies both from 
http://api.openweathermap.org/data/2.5/ as well as http://freegeoip.net/json/
- Now making use of type safe enum for temperature measurement instead of 
String.
- Changed the Consumer to not set the 'query' as {{Property}} on the current 
exchange *BUT* as the {{Header}} of the In Message (as it's the common case by 
Camel components)
- Introduced {{WeatherConstants}}.
- Some other polishments here & there.
- Updated Wiki to reflect some of these changes.

Last but not least a bunch of thanks to Rob for contributing this cool & handy 
Camel component.

BTW Rob the location being generated *is* accurate.
                
> Weather component
> -----------------
>
>                 Key: CAMEL-6398
>                 URL: https://issues.apache.org/jira/browse/CAMEL-6398
>             Project: Camel
>          Issue Type: New Feature
>            Reporter: Rob Davies
>            Assignee: Rob Davies
>             Fix For: 2.12.0
>
>
> After 6 years of lousy weather in the UK - it would be nice to get 
> notifications of good weather coming! A weather component could use the open 
> weather map API to get current weather and forecasts

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to