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

Add Allies08b #15583

Merged
merged 1 commit into from Dec 23, 2018

Conversation

Projects
None yet
7 participants
@Smittytron
Copy link
Contributor

Smittytron commented Sep 2, 2018

The scripts for Allies08a(#15582) and Allies08b are identical sans waypoints. There's a one line difference as Allies08a had one more beach to land on than Allies08b.

Using the timer conversion math from #15396 I had the mission timer at 27 min from the original, which had a 45 min timer. This seemed a bit long in testing, so I decided to shorten it to 21 min.

The player can also win early by destroying all Soviet forces on the map.

@Smittytron Smittytron referenced this pull request Sep 2, 2018

Merged

Add Allies08a #15582

@Smittytron Smittytron force-pushed the Smittytron:Allies08b branch from be2a8c9 to 27881d7 Sep 6, 2018

@abcdefg30

This comment has been minimized.

Copy link
Member

abcdefg30 commented Oct 7, 2018

Allies08a was merged.

@Smittytron Smittytron force-pushed the Smittytron:Allies08b branch from 27881d7 to 8a8b7aa Oct 7, 2018

@Smittytron

This comment has been minimized.

Copy link
Contributor

Smittytron commented Oct 7, 2018

Updated

@obrakmann obrakmann referenced this pull request Oct 8, 2018

Open

Add Red Alert campaign #4989

6 of 8 tasks complete
@abcdefg30
Copy link
Member

abcdefg30 left a comment

The reinforcements never move to attack the allied base but patrol their approach path instead.

@Muh-Muh

This comment has been minimized.

Copy link
Contributor

Muh-Muh commented Oct 10, 2018

The Evacuation objective didn't fire for me. The Scientists changed their owner as expected but just stood near the flare (where I ordered them to go earlier). It might have been caused by some mines I accidently placed at the start of the mission. Perhaps one was on the MCVEntry.location or blocked the path or something.

@abcdefg30

This comment has been minimized.

Copy link
Member

abcdefg30 commented Oct 11, 2018

Yes, they won't move if the path is blocked.

@Smittytron Smittytron force-pushed the Smittytron:Allies08b branch 2 times, most recently from 9e95e4a to da490ed Oct 29, 2018

@Smittytron

This comment has been minimized.

Copy link
Contributor

Smittytron commented Oct 29, 2018

The reinforcements never move to attack the allied base but patrol their approach path instead.

Fixed the other point, but haven't figured out the patrol thing yet. (It seems to work for everything but the south path) I'll keep trying to figure it out.

@Smittytron Smittytron force-pushed the Smittytron:Allies08b branch from da490ed to fc39233 Oct 29, 2018

@Smittytron

This comment has been minimized.

Copy link
Contributor

Smittytron commented Oct 29, 2018

@abcdefg30 Moved the waypoint as suggested in IRC. Didn't see any units going backwards in my play-through.

PR updated

@ManuGithubSteam

This comment has been minimized.

Copy link

ManuGithubSteam commented Nov 15, 2018

Why is this not merged ?

@abcdefg30

This comment has been minimized.

Copy link
Member

abcdefg30 commented Nov 15, 2018

Why is this not merged ?

I haven't retested the PR after the last set of changes.

@Smittytron Smittytron force-pushed the Smittytron:Allies08b branch from fc39233 to 26adc97 Nov 25, 2018

@Smittytron

This comment has been minimized.

Copy link
Contributor

Smittytron commented Nov 25, 2018

Rebased

@MustaphaTR
Copy link
Member

MustaphaTR left a comment

Looks good to me.

@pchote pchote added the PR: Needs +2 label Dec 23, 2018

@obrakmann obrakmann merged commit 65269f7 into OpenRA:bleed Dec 23, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@obrakmann

This comment has been minimized.

Copy link
Contributor

obrakmann commented Dec 23, 2018

@Smittytron Smittytron deleted the Smittytron:Allies08b branch Dec 24, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment