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

Cruise missile test #3243

Open
wants to merge 4 commits into
base: unstable
Choose a base branch
from

Conversation

wersal454
Copy link

@wersal454 wersal454 commented May 1, 2024

What type of PR is this.

  1. Bug
  2. Change
  3. Enhancement
  4. Test

What have you changed and why?

Information:

Used SAM code to make cruise missile work. Currently follows the target but I think it's really easy to avoid by placing either laser or a sphere on the target position and then fire at laser/sphere

Please verify the following and ensure all checks are completed.

  1. Have you loaded the mission in LAN host?
  2. Have you loaded the mission on a dedicated server?

Is further testing or are further changes required?

  1. No
  2. Yes (Please provide further detail below.)

How can the changes be tested?

Steps:
["CRUISEMISSILE", Occupants, "attack", 125, player, getPosATL player, 1, 0] spawn A3A_fnc_createSupport;


Notes:
used getPosATL player, but I think the same thing will happen if it would be getPosATL vehicle player

@wersal454 wersal454 marked this pull request as draft May 1, 2024 17:51
@wersal454
Copy link
Author

Could use same code in cruise missile functions , but it gave me error in fn_createSupport line 54
private _createFunc = missionNamespace getVariable ("A3A_fnc_SUP_" + _type);

@wersal454 wersal454 changed the title Patch 16 Cruise missile test May 1, 2024
@wersal454
Copy link
Author

It works in it's own function now, me dummy decided to rename support to "MISSILE" and it bricked stuff

@wersal454 wersal454 marked this pull request as ready for review May 5, 2024 20:51
@Bob-Murphy Bob-Murphy added this to the Not organized yet milestone Jun 8, 2024
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.

3 participants