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

Fix bike shop variant's roof #51729

Merged
merged 2 commits into from
Sep 23, 2021

Conversation

SegaSaturnity
Copy link
Contributor

Summary

None

Purpose of change

One of the bike shop variants (s_bike_shop_1) has a ladder going up on the main floor but no ladder on the roof. In fact, it has an empty roof.

You can reproduce it by making a new world, a new character, debug reveal the map and teleporting into several bike shops, the one with the open floor plan has this problem.

Describe the solution

Fixed the small typo on bike_shop.json so the roof is generated properly. Also changed the stairs of the roof into a ladder, like the main floor.

Describe alternatives you've considered

Testing

Made a new character, debug revealed map and teleported into several bike shops, all have normal roofs now.

Additional context

Before the change - empty roof
image

After the change - normal roof
image

Fix the bike shop variant 1 having an empty roof
Changed stairs down to ladder down
@actual-nh actual-nh added <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. Map / Mapgen Overmap, Mapgen, Map extras, Map display labels Sep 20, 2021
@kevingranade kevingranade merged commit 79e3e04 into CleverRaven:master Sep 23, 2021
@SegaSaturnity SegaSaturnity deleted the bikeshop_fix branch September 23, 2021 17:35
@Termineitor244
Copy link
Contributor

You should add a migration if you are going to change an ID of something already in the maps of players...

It causes an error in already started games where the building (with the original ID) is around.

imagen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants