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

Allow the teamster's mission to place the Exodii base #53000

Merged
merged 1 commit into from
Nov 25, 2021

Conversation

Venera3
Copy link
Member

@Venera3 Venera3 commented Nov 24, 2021

Summary

None

Purpose of change

Allow the Exodii base's breadcrumb mission to place the special, instead of just trying to passively find it.

Describe the solution

Added an om_special entry to the mission target array, otherwise it doesn't know which overmap special should be placed to spawn the tile in question.

Describe alternatives you've considered

Testing

Changed the base's occurrences to 0, 0, took the mission, base still spawned (meaning it's hooked up to the "turn the special mandatory" function, and got placed by the mission itself instead of random chance).

Additional context

Just a quickie fix for something that came up on Discord

@Venera3 Venera3 requested a review from I-am-Erk as a code owner November 24, 2021 20:22
@ghost
Copy link

ghost commented Nov 24, 2021

It would be good if the scout mission received from refugee center mission chain also behaves like that. Also teamster can fail the same way with finding the hub 01.

Related: #52643

@Maleclypse Maleclypse added <Bugfix> This is a fix for a bug (or closes open issue) Spawn Creatures, items, vehicles, locations appearing on map Map / Mapgen Overmap, Mapgen, Map extras, Map display and removed Spawn Creatures, items, vehicles, locations appearing on map labels Nov 25, 2021
@kevingranade kevingranade merged commit b2da387 into CleverRaven:master Nov 25, 2021
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) Map / Mapgen Overmap, Mapgen, Map extras, Map display
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants