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

Add Seaports and Airbases to Jam Script #2249

Merged
merged 1 commit into from Feb 13, 2022

Conversation

Lazejun
Copy link

@Lazejun Lazejun commented Feb 13, 2022

What type of PR is this.

  1. Bug
  2. Change
  3. Enhancement

What have you changed and why?

Information:
The Jam script only included Outposts, this could cause confusion to Players as they expected Airbases to also Own Radiotowers but they don't.
Seaports are basicly Outposts just with Water so it makes sense to include them.

Please specify which Issue this PR Resolves.

closes #2248

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:
Capture Seaports/Airports with Radiotowers nearby,
The Jam script should lower its values/set them to 1 when there is no other Tower Nearby.
Removed the Line for PVP
Possible Maps to check with that:
CLN,
Sahrani

As it Tested with Sahrani:

tierwar = 10;
//Occupants
[Teamplayer, "airport_2"] remoteExec ["A3A_fnc_markerChange", 2];
player allowdamage false;
player setpos [9856.02,10056.1,0];

Check the Jam applying: 1 is nothing applied

player getVariable "tf_sendingDistanceMultiplicator"; //50 to 1

player getVariable "tf_receivingDistanceMultiplicator"; //0.2 to 1

If you test before the changes the nearest Outpost at "airport_2" was "outpost_4".


Notes:

@Lazejun Lazejun added the Change A change to existing functionality label Feb 13, 2022
@Lazejun Lazejun added this to the 2.5.X milestone Feb 13, 2022
@Lazejun Lazejun self-assigned this Feb 13, 2022
@Bob-Murphy Bob-Murphy merged commit ff9ec94 into official-antistasi-community:unstable Feb 13, 2022
@killerswin2
Copy link

airportsX not airportsx

@Bob-Murphy Bob-Murphy added the Added to changelog Added to changelog label Dec 1, 2022
@Lazejun Lazejun deleted the Jamtime branch January 7, 2023 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Added to changelog Added to changelog Change A change to existing functionality Ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Change]: Airports and Seaports are not included in the TFAR Jam Script
3 participants