Cataclysmanic / GO2022

4 stars 1 forks source link

Buildings are copies instead of instances #81

Closed plexsoup closed 1 year ago

plexsoup commented 2 years ago

See if we can change the buildings so they're instantiated from BuildingTemplate.tscn instead of being complete copies of BuildingTemplate.

Currently, they're full copies of BuildingTemplate, which loses the benefits of scene instantiation with export variables. It also means we have to change each of the building scenes whenever we make a change to the parent (outside of the shared script)

They'd need exported variables for textures: walls, ground, etc. They'd need some warnings if they expect certain things to be present (eg: NPC spawn locations and Quest Objective markers.)

plexsoup commented 1 year ago

Not worth fixing.