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

RA - Add Allies07 #15542

Merged
merged 1 commit into from Sep 27, 2018

Conversation

Projects
None yet
4 participants
@Smittytron
Copy link
Contributor

Smittytron commented Aug 19, 2018

My goal for difficulty here is to have an easy mode, a standard fight the enemy mode for Normal difficulty, and a bit of oomph for hard difficulty. (In the way of 30 tanks dropping on the player at once)

I've reassessed my former goal of adding a 'Real Tough Guy' fourth difficulty to each mission. I now believe that those only fit in a way where they completely alter how the mission is played. Allies02 and Allies05a are golden examples of how this should be done. I couldn't think of a way to change how this mission plays out, therefore, three difficulty levels.

@Smittytron Smittytron force-pushed the Smittytron:Allies07 branch 6 times, most recently from e73a47f to f58154c Aug 19, 2018

@Smittytron Smittytron force-pushed the Smittytron:Allies07 branch from f58154c to d2324c3 Sep 2, 2018

@Orb370
Copy link

Orb370 left a comment

Played through the mission on normal difficulty with a few restarts. No issues.

@SoScared
Copy link
Member

SoScared left a comment

Played Normal and Hard, encountered no issues.

I feel Normal might a bit too strong at the beginning, baiting v2 shots, heavy tanks, yaks and careful targeting. Could be a high barrier for some.

@Smittytron Smittytron dismissed stale reviews from SoScared and Orb370 via c2b809e Sep 17, 2018

@Smittytron Smittytron force-pushed the Smittytron:Allies07 branch 2 times, most recently from c2b809e to 5b2ae90 Sep 17, 2018

@Smittytron

This comment has been minimized.

Copy link
Contributor

Smittytron commented Sep 17, 2018

Updated. Removed v2 production from easy and normal, and renamed waypoints per IRC nit.

@SoScared
Copy link
Member

SoScared left a comment

Mission replayed on Normal. Feels well paced for its difficulty level.

@abcdefg30
Copy link
Member

abcdefg30 left a comment

Looks ok to me otherwise.


InitialAlliedReinforcements = function()
Trigger.AfterDelay(DateTime.Seconds(1), function()
Reinforcements.Reinforce(greece, AlliedGunboats, { GunboatEntry.Location, waypoint42.Location }, 2)

This comment has been minimized.

@abcdefg30

abcdefg30 Sep 19, 2018

Member

Missing one level of indentation here.

ussr.Cash = 5000
badguy.Cash = 500

if StartTimer == true then

This comment has been minimized.

@abcdefg30

abcdefg30 Sep 19, 2018

Member

if StartTimer then

WorldLoaded = function()
greece = Player.GetPlayer("Greece")
ussr = Player.GetPlayer("USSR")
england = Player.GetPlayer("England")

This comment has been minimized.

@abcdefg30

abcdefg30 Sep 19, 2018

Member

This appears to be unused.

ParatroopersPower:
DropItems: E1,E1,E1,E2,E2


This comment has been minimized.

@abcdefg30

abcdefg30 Sep 19, 2018

Member

Double new line.

LuaScript:
Scripts: allies07.lua, allies07-AI.lua
MissionData:
Briefing: Briefing: LANDCOM 16 HQS.@TOP SECRET.@TO: FIELD COMMANDER A9@@INTERCEPTION OF SOVIET COMMUNIQUE INDICATES THEIR IRON CURTAIN RESEARCH WAS SET BACK BY ESPIONAGE. EXCELLENT WORK, COMMANDER!@@COMMUNIQUE WAS TRACED BACK TO SECRET SOVIET BASE IN BORNHOLM. INVESTIGATE POSSIBLE CONNECTION WITH IRON CURTAIN RESEARCH. CAPTURE RADAR CENTER AND DESTROY SUB PRODUCTION CAPABILITY.@@CONFIRMATION CODE 1138.@@TRANSMISSION ENDS.

This comment has been minimized.

@abcdefg30

abcdefg30 Sep 19, 2018

Member

Please replace all @s by new lines (\n).

StartTimerFunction = function()
if Map.LobbyOption("difficulty") == "hard" then
StartTimer = true
end

This comment has been minimized.

@abcdefg30

abcdefg30 Sep 19, 2018

Member

Can we get a "Timer started" notification here?

}

SendArmoredBattalion = function()
Utils.Do(BattalionWays, function(way)

This comment has been minimized.

@abcdefg30

abcdefg30 Sep 19, 2018

Member

The "Enemy units approaching" speech notification would be nice here.

@Smittytron Smittytron force-pushed the Smittytron:Allies07 branch 3 times, most recently from 12825e8 to 46de2bd Sep 20, 2018

@Smittytron

This comment has been minimized.

Copy link
Contributor

Smittytron commented Sep 20, 2018

Updated

LuaScript:
Scripts: allies07.lua, allies07-AI.lua
MissionData:
Briefing: Briefing: LANDCOM 16 HQS.\nTOP SECRET.\nTO: FIELD COMMANDER A9\n\nINTERCEPTION OF SOVIET COMMUNIQUE INDICATES THEIR IRON CURTAIN RESEARCH WAS SET BACK BY ESPIONAGE. EXCELLENT WORK, COMMANDER!\n\nCOMMUNIQUE WAS TRACED BACK TO SECRET SOVIET BASE IN BORNHOLM. INVESTIGATE POSSIBLE CONNECTION WITH IRON CURTAIN RESEARCH. CAPTURE RADAR CENTER AND DESTROY SUB PRODUCTION CAPABILITY.\n\nCONFIRMATION CODE 1138.\n\nTRANSMISSION ENDS.

This comment has been minimized.

@abcdefg30

abcdefg30 Sep 26, 2018

Member

Duplicate Briefing:.
Please add a \n at the end, as well.

@Smittytron Smittytron force-pushed the Smittytron:Allies07 branch from 46de2bd to 57d3e67 Sep 26, 2018

@Smittytron

This comment has been minimized.

Copy link
Contributor

Smittytron commented Sep 26, 2018

Updated

@abcdefg30 abcdefg30 merged commit 9fb8f6c into OpenRA:bleed Sep 27, 2018

2 checks passed

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

This comment has been minimized.

Copy link
Member

abcdefg30 commented Sep 27, 2018

@Smittytron Smittytron deleted the Smittytron:Allies07 branch Sep 28, 2018

@obrakmann obrakmann referenced this pull request Oct 8, 2018

Open

Add Red Alert campaign #4989

6 of 8 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment