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

Adding function addVehicleSirens & removeVehicleSirens client-side #3704

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

Proxy-99
Copy link
Contributor

@Proxy-99 Proxy-99 commented Sep 7, 2024

closing #3215
added removeVehicleSirens() and addVehicleSirens()

test with client script

local vehicle = getPedOccupiedVehicle(localPlayer)

   local result =    addVehicleSirens(vehicle, 2, 3,false,true,true,false)
      setVehicleSirens(vehicle, 1, -0.3, 0, 0.8, 0, 0, 255, 255, 255)
      setVehicleSirens(vehicle, 2, 0.3, 0, 0.8, 255, 0, 0, 255, 255)
     setVehicleSirensOn(vehicle, true)

     print(result)

function remove ()
      removeVehicleSirens (vehicle)
end

     addCommandHandler("remove", remove)

@Proxy-99 Proxy-99 marked this pull request as ready for review September 8, 2024 14:46
@Proxy-99 Proxy-99 changed the title Adding function addVehicleSirens client-side Adding function addVehicleSirens & removeVehicleSirens client-side Sep 8, 2024
@Proxy-99 Proxy-99 requested a review from FileEX September 8, 2024 17:09
@tederis tederis added the enhancement New feature or request label Sep 10, 2024
@Fernando-A-Rocha
Copy link
Contributor

What can we do about #1375?

@Fernando-A-Rocha
Copy link
Contributor

@Proxy-99 with this feature we will be able to add sirens to engineRequestModel IDs clientside? Like a vehicle with parent ID Police LS.

@Proxy-99
Copy link
Contributor Author

Proxy-99 commented Sep 13, 2024

@Proxy-99 with this feature we will be able to add sirens to engineRequestModel IDs clientside? Like a vehicle with parent ID Police LS.

After I test it with engineRequestModel it won't works it just the sound of the siren that works similar like trains ids

tederis
tederis previously approved these changes Sep 14, 2024
@Fernando-A-Rocha
Copy link
Contributor

@Proxy-99 what's the state of this?😀

@Proxy-99
Copy link
Contributor Author

@Proxy-99 what's the state of this?😀

need someone to merge it

@Fernando-A-Rocha
Copy link
Contributor

i'm gonna test.

TheNormalnij
TheNormalnij previously approved these changes Oct 13, 2024
TheNormalnij
TheNormalnij previously approved these changes Oct 15, 2024
Copy link
Contributor

@Fernando-A-Rocha Fernando-A-Rocha left a comment

Choose a reason for hiding this comment

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

OK to merge but should fix vehicle sirens not working on engineRequestModel ids on another PR

@Proxy-99
Copy link
Contributor Author

@TheNormalnij @tederis can u merge this

@TheNormalnij
Copy link
Member

@TheNormalnij @tederis can u merge this

Our build server doesn't work right now, and the next build is big enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants