Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add oldwest-style buildings #89

Merged
merged 41 commits into from
Oct 1, 2021
Merged

Conversation

jdrueckert
Copy link
Member

@jdrueckert jdrueckert commented Aug 3, 2020

@skaldarnar and @jdrueckert just started building some old-west styled buildings for the fun of it. For now, we'll dump the structure templates in this draft PR.

As discussed on Discord, they'll eventually be ending up in themed module OldWestCities (naming up for discussion, this one would be in line with MedievalCities.

Depends on Terasology/DynamicCities#105

TODO:

  • open doors in residential house 3
  • fix missing blocks due to template utility block

@jdrueckert jdrueckert added the Type: Improvement Request for or addition/enhancement of a feature label Aug 3, 2020
@jdrueckert jdrueckert self-assigned this Aug 3, 2020
@skaldarnar
Copy link
Contributor

Updated issue description.

Comment on lines 5 to 6
"MetalRenegades:oldwestResidentialHouse1",
"MetalRenegades:oldwestResidentialHouse2"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will place both buildings on top of each other, one after the other. See Terasology/DynamicCities#104 where I've added some docstrings to clarify this behavior.

@jdrueckert jdrueckert marked this pull request as ready for review October 1, 2021 19:07
@jdrueckert jdrueckert changed the title feat: add oldwest-style residential houses feat: add oldwest-style buildings Oct 1, 2021
@jdrueckert jdrueckert merged commit cac50b8 into develop Oct 1, 2021
@jdrueckert jdrueckert deleted the feat/oldwest-buildings branch October 1, 2021 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Improvement Request for or addition/enhancement of a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants