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

Mainline jsonified fema camp from the Mapgen Demo mod #34081

Merged
merged 2 commits into from
Sep 17, 2019
Merged

Mainline jsonified fema camp from the Mapgen Demo mod #34081

merged 2 commits into from
Sep 17, 2019

Conversation

Night-Pryanik
Copy link
Contributor

Summary

SUMMARY: Content "Mainline jsonified fema camp from the Mapgen Demo mod."

Purpose of change

Mainline jsonified fema camp from the Mapgen Demo mod.

Describe the solution

Moved overmap terrain, overmap special, pallette from the mod to mainline.
Also removed some unused symbols from pallette.
Also decreased chance to spawn from 1..5 to 0..5 for both versions of FEMA Camp. I think at least one guaranteed spawn of the camp on the overmap is too much.
Also fixed road connection for the camp.

I hesitated to fully replace hardcoded version with this jsonified version as they are way too different. So for now there are two differing versions of the camp - jsonified one and hardcoded one. We can always jsonify hardcoded version later.

@Night-Pryanik Night-Pryanik added <Enhancement / Feature> New features, or enhancements on existing Mods Issues related to mods or modding Map / Mapgen Overmap, Mapgen, Map extras, Map display labels Sep 17, 2019
@ZhilkinSerg ZhilkinSerg merged commit 7687f57 into CleverRaven:master Sep 17, 2019
@Night-Pryanik Night-Pryanik deleted the mainline-jsonified-fema-camp branch September 17, 2019 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Enhancement / Feature> New features, or enhancements on existing Map / Mapgen Overmap, Mapgen, Map extras, Map display Mods Issues related to mods or modding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants