← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/bug-1687100-reveal_fields into lp:widelands

 

Looks good now :-)

One small nit: Since hide_completely and reveal_completely are optional it should be clarified which boolean (true/false) will take an effect at all.

I want to make some more tests, before approval of testing. In the meantime someone else could look at the logic...
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1687100-reveal_fields/+merge/323721
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1687100-reveal_fields.


References