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

Soviet05 reveal enemy base after capturing radar dome #14329

Merged
merged 1 commit into from Nov 26, 2017

Conversation

Projects
None yet
5 participants
@Arular101
Contributor

Arular101 commented Nov 10, 2017

A proposal to polish the Soviet-05 mission.

The first objective in this mission is to capture the enemy radar dome. Unfortunately, like in the original game, nothing happens. Although, the mission briefing says: Capture their radar center so we can track Allied activity in this area.

I think a nice feature would be to reveal the enemy base after the radar dome is captured.

I decided to reveal the base with a big radius on easy difficulty level. On normal and hard, a small radius is used. It only shows the MCVs location.

@abcdefg30

This comment has been minimized.

Show comment
Hide comment
@abcdefg30

abcdefg30 Nov 11, 2017

Member

Good idea! What do you think about placing a beacon with a text message hinting the player to what is going on (e.g. "We are now able to track allied movement in this area and discovered there expansion base.").

Member

abcdefg30 commented Nov 11, 2017

Good idea! What do you think about placing a beacon with a text message hinting the player to what is going on (e.g. "We are now able to track allied movement in this area and discovered there expansion base.").

@Arular101

This comment has been minimized.

Show comment
Hide comment
@Arular101

Arular101 Nov 12, 2017

Contributor

Updated.

Yeah, that is a great feature! I added the beacon and message. I hope the message is good.

I also did it in Allies 06a, see PR #14328.

Contributor

Arular101 commented Nov 12, 2017

Updated.

Yeah, that is a great feature! I added the beacon and message. I hope the message is good.

I also did it in Allies 06a, see PR #14328.

@Arular101

This comment has been minimized.

Show comment
Hide comment
@Arular101

Arular101 Nov 19, 2017

Contributor

Changed the message as requested at #14328 .

Contributor

Arular101 commented Nov 19, 2017

Changed the message as requested at #14328 .

@Arular101

This comment has been minimized.

Show comment
Hide comment
@Arular101

Arular101 Nov 20, 2017

Contributor

Apparently somehow the review of reaperrr was dismissed by me. I don’t know how that happen, but I didn’t change anything after the review. So, I consider reaperrrs review still valid.

Contributor

Arular101 commented Nov 20, 2017

Apparently somehow the review of reaperrr was dismissed by me. I don’t know how that happen, but I didn’t change anything after the review. So, I consider reaperrrs review still valid.

@Smittytron

This comment has been minimized.

Show comment
Hide comment
@Smittytron

Smittytron Nov 20, 2017

Contributor

Not a big deal, but I'd go with something like "Allied MCV detected moving to the island." or "The Allied MCV is fleeing to the island!" Then, "Coordinates of new Allied base discovered."

👍 After squashing.

Contributor

Smittytron commented Nov 20, 2017

Not a big deal, but I'd go with something like "Allied MCV detected moving to the island." or "The Allied MCV is fleeing to the island!" Then, "Coordinates of new Allied base discovered."

👍 After squashing.

@Arular101

This comment has been minimized.

Show comment
Hide comment
@Arular101

Arular101 Nov 23, 2017

Contributor

That the MCV is moving to the island is in a different trigger. I agree that it is a good idea to show a message when the MCV is moving, because this could happen in different moments.

Just to clarify the current state: on easy, the whole expansion base is shown with the message "Movement of an Allied expansion base discovered."

On normal and hard, only the construction yard is shown with the message "Coordinates of an Allied expansion base discovered."

So, I'll add the message "Allied MCV detected moving to the island." for when the MCV is triggered to move. (By the way, this happens at the start at hard difficulty level.)

Contributor

Arular101 commented Nov 23, 2017

That the MCV is moving to the island is in a different trigger. I agree that it is a good idea to show a message when the MCV is moving, because this could happen in different moments.

Just to clarify the current state: on easy, the whole expansion base is shown with the message "Movement of an Allied expansion base discovered."

On normal and hard, only the construction yard is shown with the message "Coordinates of an Allied expansion base discovered."

So, I'll add the message "Allied MCV detected moving to the island." for when the MCV is triggered to move. (By the way, this happens at the start at hard difficulty level.)

Soviet05 Radar Reveal
Added beacon and message
@Arular101

This comment has been minimized.

Show comment
Hide comment
@Arular101

Arular101 Nov 24, 2017

Contributor

Updated. Moved the RunInitialActivities() a little lower so on hard difficulty the message of the MCV moving to the island is shown below the mission objectives.

Just noticed that on bleed one of the AA Guns is in low power. But I can't find the reason why that is.
openra-2017-11-24t151034396z

Contributor

Arular101 commented Nov 24, 2017

Updated. Moved the RunInitialActivities() a little lower so on hard difficulty the message of the MCV moving to the island is shown below the mission objectives.

Just noticed that on bleed one of the AA Guns is in low power. But I can't find the reason why that is.
openra-2017-11-24t151034396z

@MustaphaTR

This comment has been minimized.

Show comment
Hide comment
@MustaphaTR

MustaphaTR Nov 24, 2017

Member

Just noticed that on bleed one of the AA Guns is in low power. But I can't find the reason why that is.

May be same reason why Outpost on Har5 is on Low Power.

Member

MustaphaTR commented Nov 24, 2017

Just noticed that on bleed one of the AA Guns is in low power. But I can't find the reason why that is.

May be same reason why Outpost on Har5 is on Low Power.

@reaperrr

This comment has been minimized.

Show comment
Hide comment
@reaperrr

reaperrr Nov 26, 2017

Contributor

Apparently somehow the review of reaperrr was dismissed by me. I don’t know how that happen, but I didn’t change anything after the review. So, I consider reaperrrs review still valid.

Don't worry, github does that automatically when updating a PR.

Contributor

reaperrr commented Nov 26, 2017

Apparently somehow the review of reaperrr was dismissed by me. I don’t know how that happen, but I didn’t change anything after the review. So, I consider reaperrrs review still valid.

Don't worry, github does that automatically when updating a PR.

@reaperrr reaperrr merged commit 02d57c5 into OpenRA:bleed Nov 26, 2017

2 checks passed

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

This comment has been minimized.

Show comment
Hide comment
@reaperrr
Contributor

reaperrr commented Nov 26, 2017

@Arular101 Arular101 deleted the Arular101:Soviet05Radar branch Nov 26, 2017

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