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

missions: autoset monster_kill_goal for KILL_TYPE and KILL_SPEC #27050

Merged
merged 1 commit into from
Dec 12, 2018

Conversation

mlangsdorf
Copy link
Contributor

@mlangsdorf mlangsdorf commented Dec 10, 2018

Summary

SUMMARY: Infrastructure "missions: autoset monster_kill_goal for KILL_TYPE and KILL_SPEC"

Purpose of change

Make it easier to add new missions in JSON, by allowing MGOAL_KILL_MONSTER_TYPE and MGOAL_KILL_MONSTER_SPEC missions to be set up almost entirely in JSON.

Describe the solution

Add monster_type as a JSON parameter, and then automatically set the kill goal for MGOAL_KILL_MONSTER_TYPE and MGOAL_KILL_MONSTER_SPEC instead of requiring a start function to do it.

Describe alternatives you've considered

This is really straightforward.

Additional context

This is a small part of a general effort to move more of mission definitions into JSON - see #26794

@mlangsdorf mlangsdorf added <Enhancement / Feature> New features, or enhancements on existing [JSON] Changes (can be) made in JSON Missions Quests and missions [C++] Changes (can be) made in C++. Previously named `Code` labels Dec 10, 2018
src/mission.h Outdated Show resolved Hide resolved
Add monster_type as a JSON parameter, and then automatically set the
kill goal for MGOAL_KILL_MONSTER_TYPE and MGOAL_KILL_MONSTER_SPEC
instead of requiring a start function to do it.
@mlangsdorf mlangsdorf force-pushed the missionstart_autokill branch from e168645 to 2e56ad6 Compare December 10, 2018 21:25
@Night-Pryanik
Copy link
Contributor

Night-Pryanik commented Dec 11, 2018

@mlangsdorf thanks for constantly improving my amateurish coding. Good job!

@ZhilkinSerg ZhilkinSerg merged commit 06c9e43 into CleverRaven:master Dec 12, 2018
@mlangsdorf mlangsdorf deleted the missionstart_autokill branch December 13, 2018 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` <Enhancement / Feature> New features, or enhancements on existing [JSON] Changes (can be) made in JSON Missions Quests and missions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants