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

Rework objectives in Situation Critical #19273

Merged
merged 1 commit into from
Apr 3, 2021
Merged

Conversation

Smittytron
Copy link
Member

Closes #19271. I didn't account for the fact that missile subs have more range in OpenRA than the original. The choice was to reduce it to 14 for this mission or change the objectives to account for multiple win scenarios, which I think is cleaner.

I also realized the original mission accounted for destroying the SAM sites before infiltrating the lab, so I adjusted the script to allow for that as well.

Copy link
Contributor

@reaperrr reaperrr left a comment

Choose a reason for hiding this comment

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

Untested, but script changes look fine

Copy link
Member

@pchote pchote left a comment

Choose a reason for hiding this comment

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

Tested and script changes look fine for now.

I raised a question about IsObjectiveCompleted versus a separate variable on Discord, but it sounds like this may be an issue across multiple missions so can be left to a followup PR.

@pchote pchote merged commit d95c414 into OpenRA:bleed Apr 3, 2021
@Smittytron Smittytron deleted the sit-crit-obj branch April 3, 2021 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Script error in situation-critical: playtest-20210131
3 participants