Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands-website/anti_spam_4 into lp:widelands-website

2016-11-14 Thread kaputtnik
Thanks :-) Merged, deployed and added the management command to /etc/cron.daily/django_regular_commands -- https://code.launchpad.net/~widelands-dev/widelands-website/anti_spam_4/+merge/309642 Your team Widelands Developers is subscribed to branch lp:widelands-website. _

[Widelands-dev] [Merge] lp:~widelands-dev/widelands-website/anti_spam_4 into lp:widelands-website

2016-11-14 Thread noreply
The proposal to merge lp:~widelands-dev/widelands-website/anti_spam_4 into lp:widelands-website has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands-website/anti_spam_4/+merge/309642 -- Your team Widelands Developers is

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/fsmenu_fullscreen_3_mapselect into lp:widelands

2016-11-14 Thread GunChleoc
GunChleoc has proposed merging lp:~widelands-dev/widelands/fsmenu_fullscreen_3_mapselect into lp:widelands. Commit message: Map Selection screen now relayouts itself on fullscreen toggle. Requested reviews: Widelands Developers (widelands-dev) Related bugs: Bug #1398733 in widelands: "Fullsc

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/fsmenu_fullscreen_3_mapselect into lp:widelands

2016-11-14 Thread GunChleoc
This is ready now. The change affects map selection, editor savemap and anything that contains a table. -- https://code.launchpad.net/~widelands-dev/widelands/fsmenu_fullscreen_3_mapselect/+merge/310744 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/casern_workersqueue into lp:widelands

2016-11-14 Thread bunnybot
Continuous integration builds have changed state: Travis build 1608. State: failed. Details: https://travis-ci.org/widelands/widelands/builds/175798398. Appveyor build 1446. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_casern_wor

[Widelands-dev] [Merge] lp:~widelands-dev/widelands-website/auto_one_to_one_ractivating into lp:widelands-website

2016-11-14 Thread kaputtnik
The proposal to merge lp:~widelands-dev/widelands-website/auto_one_to_one_ractivating into lp:widelands-website has been updated. Description changed to: Another issue from the update to django 1.8: Formerly to django 1.8 entrys to the database of wlprofile and wlggz has been created on deman

[Widelands-dev] [Merge] lp:~widelands-dev/widelands-website/auto_one_to_one_ractivating into lp:widelands-website

2016-11-14 Thread kaputtnik
kaputtnik has proposed merging lp:~widelands-dev/widelands-website/auto_one_to_one_ractivating into lp:widelands-website. Requested reviews: Widelands Developers (widelands-dev) For more details, see: https://code.launchpad.net/~widelands-dev/widelands-website/auto_one_to_one_ractivating/+mer

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1624216-horsepocalypse into lp:widelands

2016-11-14 Thread bunnybot
Continuous integration builds have changed state: Travis build 1607. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/175655541. Appveyor build 1445. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_162421

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1624216-horsepocalypse into lp:widelands

2016-11-14 Thread bunnybot
Bunnybot encountered an error while working on this merge proposal: HTTP Error 500: Internal Server Error -- https://code.launchpad.net/~widelands-dev/widelands/bug-1624216-horsepocalypse/+merge/310221 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/casern_workersqueue into lp:widelands

2016-11-14 Thread GunChleoc
I am always in favour of removing code duplication wherever possible, so please go ahead if you want. If it's older code, you can also do that in a follow-up branch to keep the current diff smaller. Regarding the builder, we actually have an open bug for that, so yes, please :) https://bugs.lau

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1624216-horsepocalypse into lp:widelands

2016-11-14 Thread GunChleoc
Thanks for the review, the code looks much better now :) There is a savegame attached to the bug that you can use for testing - attack the headquarters to the north. -- https://code.launchpad.net/~widelands-dev/widelands/bug-1624216-horsepocalypse/+merge/310221 Your team Widelands Developers is

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/scrollbar_beautification into lp:widelands

2016-11-14 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/scrollbar_beautification into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/scrollbar_beautification/+merge/309635 -- Your team Widelands Developers i

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/fsmenu_fullscreen_3_mapselect into lp:widelands

2016-11-14 Thread GunChleoc
The proposal to merge lp:~widelands-dev/widelands/fsmenu_fullscreen_3_mapselect into lp:widelands has been updated. Status: Needs review => Work in progress For more details, see: https://code.launchpad.net/~widelands-dev/widelands/fsmenu_fullscreen_3_mapselect/+merge/310744 -- Your team Wi

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/fsmenu_fullscreen_3_mapselect into lp:widelands

2016-11-14 Thread GunChleoc
Scrollbar beautification branch just got approved, so putting this on hold for the TODO. -- https://code.launchpad.net/~widelands-dev/widelands/fsmenu_fullscreen_3_mapselect/+merge/310744 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/fsme

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/scrollbar_beautification into lp:widelands

2016-11-14 Thread GunChleoc
Yes, that's on purpose. We need new art. https://wl.widelands.org/forum/topic/2648/ Thanks for the review @bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/scrollbar_beautification/+merge/309635 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widel

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/fsmenu_fullscreen_3_mapselect into lp:widelands

2016-11-14 Thread GunChleoc
The proposal to merge lp:~widelands-dev/widelands/fsmenu_fullscreen_3_mapselect into lp:widelands has been updated. Commit Message changed to: Map Selection screen now relayouts itself on fullscreen toggle. For more details, see: https://code.launchpad.net/~widelands-dev/widelands/fsmenu_fullsc

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/fsmenu_fullscreen_3_mapselect into lp:widelands

2016-11-14 Thread GunChleoc
GunChleoc has proposed merging lp:~widelands-dev/widelands/fsmenu_fullscreen_3_mapselect into lp:widelands. Commit message: Map Selection screen now relayouts itself on fullscreen toggle. Requested reviews: Widelands Developers (widelands-dev) Related bugs: Bug #1398733 in widelands: "Fullsc

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/casern_workersqueue into lp:widelands

2016-11-14 Thread bunnybot
Continuous integration builds have changed state: Travis build 1604. State: failed. Details: https://travis-ci.org/widelands/widelands/builds/175631314. Appveyor build 1442. State: failed. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_casern_work

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/scrollbar_beautification into lp:widelands

2016-11-14 Thread SirVer
Review: Approve lgtm. Unrelated: have you started the game at 800x600 lately? The 'Widelands' logo is partly covered by the edge images. Diff comments: > > === modified file 'src/ui_basic/table.cc' > --- src/ui_basic/table.cc 2016-10-24 14:04:00 + > +++ src/ui_basic/table.cc 2016-

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands-website/anti_spam_4 into lp:widelands-website

2016-11-14 Thread SirVer
Review: Approve lgtm. -- https://code.launchpad.net/~widelands-dev/widelands-website/anti_spam_4/+merge/309642 Your team Widelands Developers is subscribed to branch lp:widelands-website. ___ Mailing list: https://launchpad.net/~widelands-dev Post to