← Back to team overview

widelands-dev team mailing list archive

Thread
Date

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

To: mp+303493@xxxxxxxxxxxxxxxxxx
From: SirVer <SirVer@xxxxxx>
Date: Tue, 23 Aug 2016 12:45:39 -0000
In-reply-to: <20160821132036.30564.74803.launchpad@ackee.canonical.com>
Reply-to: mp+303493@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

sgtm - I agree that having the value once is preferable and the model seems the right place for it. Having a comment there why it is only needed for one of the values should be sufficient.
-- 
https://code.launchpad.net/~widelands-dev/widelands-website/move_minimaps/+merge/303493
Your team Widelands Developers is subscribed to branch lp:widelands-website.

References

[Merge] lp:~widelands-dev/widelands-website/move_minimaps into lp:widelands-website
From: kaputtnik, 2016-08-21