[Marble-devel] Review Request 108021: Migrate Weather Plugin to usage of MapInfoDialog

[Torsten Rahn]

> On March 7, 2014, 5:36 p.m., Bernhard Beschow wrote:
> > src/plugins/render/weather/WeatherItem.cpp, line 524
> > <https://git.reviewboard.kde.org/r/108021/diff/1/?file=102962#file102962line524>
> >
> >     Why is no credit given any more? Aren't we supposed to give credit by license requirements?

Good catch. I think we give credit in the Plugin about dialog but you are right that this should not be the only place.


- Torsten


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/108021/#review52370
-----------------------------------------------------------


On Dec. 30, 2012, 10:24 a.m., Illya Kovalevskyy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/108021/
> -----------------------------------------------------------
> 
> (Updated Dec. 30, 2012, 10:24 a.m.)
> 
> 
> Review request for Marble, Dennis Nienhüser, Torsten Rahn, and René Küttner.
> 
> 
> Repository: marble
> 
> 
> Description
> -------
> 
> Weather Plugin has been migrated to usage of MapInfoDialog successfully according to GCI task:
>  - https://google-melange.appspot.com/gci/task/view/google/gci2012/8097206
> 
> 
> Diffs
> -----
> 
>   src/plugins/render/weather/AbstractWeatherService.h 8c8ce0e 
>   src/plugins/render/weather/AbstractWeatherService.cpp 251fe4d 
>   src/plugins/render/weather/BBCWeatherService.cpp 02898c1 
>   src/plugins/render/weather/CMakeLists.txt 7b48dc9 
>   src/plugins/render/weather/GeoNamesWeatherService.cpp 202a9c3 
>   src/plugins/render/weather/WeatherItem.h 09829f9 
>   src/plugins/render/weather/WeatherItem.cpp ae80079 
>   src/plugins/render/weather/WeatherModel.h 7613b5f 
>   src/plugins/render/weather/WeatherModel.cpp e6580d3 
>   src/plugins/render/weather/WeatherPlugin.h 844f5b6 
>   src/plugins/render/weather/WeatherPlugin.cpp dd5e3de 
>   src/plugins/render/weather/data/marble_deco.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-clear-night.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-clear.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-clouds-night.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-clouds.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-few-clouds-night.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-few-clouds.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-freezing-rain.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-hail.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-many-clouds.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-mist.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-showers-day.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-showers-night.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-showers-scattered-day.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-showers-scattered-night.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-showers-scattered.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-showers.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-snow-rain.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-snow-scattered-day.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-snow-scattered-night.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-snow-scattered.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-snow.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-storm-day.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-storm-night.png PRE-CREATION 
>   src/plugins/render/weather/data/weather-storm.png PRE-CREATION 
>   src/plugins/render/weather/data/weather.html PRE-CREATION 
>   src/plugins/render/weather/data/weatherscreen.css PRE-CREATION 
>   src/plugins/render/weather/weather.qrc PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/108021/diff/
> 
> 
> Testing
> -------
> 
> Built & Tested with love.
> 
> 
> Screenshots
> -----------
> 
> Usecase 1
>   https://git.reviewboard.kde.org/r/108021/s/938/
> Usecase 2
>   https://git.reviewboard.kde.org/r/108021/s/939/
> Usecase 3
>   https://git.reviewboard.kde.org/r/108021/s/940/
> 
> 
> Thanks,
> 
> Illya Kovalevskyy
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20140307/17ad0e18/attachment-0001.html>