← Back to team overview

widelands-dev team mailing list archive

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

 

Review: Resubmit

Refactored everything except for:

1. has_window() and the economy merge code. Handling that with UniqueWindowHandler is not trivial, because the UniqueWindow's ID needs to change when the economy number changes. I spent quite a lot of time on it and decided to drop it, because it didn't work.

2. Merging the Shipmessages is snowballing, so better keep that for a separate branch.
-- 
https://code.launchpad.net/~widelands-dev/widelands/notifications_economy_window/+merge/311572
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/notifications_economy_window.


References