Al Mele
Al Mele
> It doesn't matter if you need updates or not, under any case you're going to have to update Minecolonies if you actually want access to the new flags, so...
> It is fine to add API for addons to interact with our systems. We already have the schematic tags though and modules that could allow for similar functionality With...
Converted to a simple getter, and smartened up the logic to look at the parent (where applicable) as well.
I'm totally open to that - I'd love to have one core dev opinion on that before I make the change, though, since I know it was received with great...
> > I think we should add a Chest icon to the building's research UI (where the 3 branches are listed below is space) which has a hover tooltip explaining...
> I'd center it and make it much bigger^^ ?
Ok, I incorporated all the code review feedback. It grew a bit in terms of code touched. I re-tested all the client paths that touched the BuildingView.getContainerList (now BuildingView.getContainers) as...
Thanks for all the feedback. When you talk about having things "on the colony level", and syncing them ourselves - could you point me to a similar pattern that already...
Just to be sure I'm not going down the wrong path here, moving the synced data elements, stable position, reservations, etc off the entity to the colony level will result...
Significantly refactored things to introduce the AnimalData/AnimalDataView pattern suggested. Code changes with this PR are now more intrusive, but will support a much nicer division of labor. Introduced the IManagedAnimal...