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

Change: Add demo charges to unused Demo Saboteur, add unique voice lines #2283

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

commy2
Copy link
Collaborator

@commy2 commy2 commented Aug 24, 2023

  • Adds previously unused voice set to the worldbuilder only Demo General Saboteur.
  • Adds Suicide ability to Demo General Saboteur
  • Adds placing timed and remote demo charges ability to Demo General Saboteur -- he has voice lines specifically for that
  • Animation used for placing charges is from CIA Agent. It looks neat with the Saboteur model.

@commy2 commy2 added WorldBuilder Affects World Builder only Minor Severity: Minor < Major < Critical < Blocker GLA Affects GLA faction Enhancement Is new feature or request labels Aug 24, 2023
@commy2
Copy link
Collaborator Author

commy2 commented Aug 25, 2023

Uh oh, Sabotage + placing demo charges is very wonky on buildings. Saboteur may collide with building and sabotage it instead of placing a bomb. Also, this allows Saboteur to target scaffolds and sabotage those. I think best is to remove infiltration from this Saboteur and make him a demolition unit entirely.

@commy2
Copy link
Collaborator Author

commy2 commented Aug 25, 2023

Camouflaged unit + suicide ability. Very sneaky.

@commy2 commy2 self-assigned this Aug 27, 2023
@MTKing4 MTKing4 self-assigned this Aug 31, 2023
Copy link
Collaborator

@xezon xezon left a comment

Choose a reason for hiding this comment

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

Tested. Looks good. Just voice create needs minor polish.

@@ -42422,6 +42422,22 @@ RU: "Саботажник: &S"
AR: "ﻱﺭﺎﺤﺘﻧﺈﻟا"
END

// Patch104p @fix commy2 25/08/2023 Adds text variant for Demo Saboteur.
Copy link
Collaborator

Choose a reason for hiding this comment

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

#2283, several times.

ZH: "非戰鬥滲透單位。 只有在他放置炸彈時會顯露出來。"
BP: "Unidade de infiltração não combatente. Visível apenas quando planta cargas de demolição."
PL: "Cywilna jednostka infiltrująca. Staje się widoczny, gdy podkłada ładunek wybuchowy."
RU: "Не боевое подразделение для проникновения. Выдает себя, когда устанавливает бомбу."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Closing Russian sentences end without dot.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Why's that?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Because Russian text follows English style.

@@ -4827,6 +4827,38 @@ AudioEvent SaboteurVoiceDie
End


; Demo Saboteur
Copy link
Collaborator

Choose a reason for hiding this comment

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

Patch104p comment missing.

@@ -17398,11 +17410,17 @@ Object Demo_GLAInfantrySaboteur
End

; ***DESIGN parameters ***
DisplayName = OBJECT:Saboteur
DisplayName = OBJECT:Demo_Saboteur // Patch104p @fix commy2 25/08/2023 Adds text variant for Demo Saboteur.
Copy link
Collaborator

Choose a reason for hiding this comment

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

#2283

@@ -2343,6 +2343,7 @@ CommandSet Demo_GLABarracksCommandSet
3 = Demo_Command_ConstructGLAInfantryTerrorist
4 = Demo_Command_ConstructGLAInfantryAngryMob
6 = Demo_Command_ConstructGLAInfantryJarmenKell
7 = Demo_Command_ConstructGLAInfantrySaboteur ; @TODO, debug remove
Copy link
Collaborator

Choose a reason for hiding this comment

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

Needs debug removal before merge, 2x.

@MTKing4 MTKing4 removed their assignment Sep 3, 2023
@@ -2703,6 +2712,7 @@ CommandSet Demo_GLABarracksCommandSetUpgrade ; unused
3 = Demo_Command_ConstructGLAInfantryTerrorist
4 = Demo_Command_ConstructGLAInfantryAngryMob
6 = Demo_Command_ConstructGLAInfantryJarmenKell
7 = Demo_Command_ConstructGLAInfantrySaboteur ; @TODO, debug remove
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do not forget to remove before commit.

Control = random
Volume = 90
Type = ui voice player
End
Copy link
Collaborator

Choose a reason for hiding this comment

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

There are also voice lines

  • isadseb "Demo Saboteur" (can be used for select)
  • isadsec "I sneak in and blow them up" (can be used for select or move)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Is new feature or request GLA Affects GLA faction Minor Severity: Minor < Major < Critical < Blocker WorldBuilder Affects World Builder only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants