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

qb-garage new feature added #311

Closed
wants to merge 9 commits into from
Closed

Conversation

Corona16
Copy link

@Corona16 Corona16 commented Nov 7, 2023

Describe Pull request
Hi i have modified little bit the qb-garages now you all the jobs have the garages with vehicles with the new interface (NUI) in qb-garages for example qb-ambulancejob does not need garage Takeoutvehicle system in the script its self as it has in qb-garages.
Potentially we can reduce the amount of coding and Threads in All the jobs just by this change!

If your PR is to fix an issue mention that issue here: none

Questions (please complete the following information):

  • Have you personally loaded this code into an updated qbcore project and checked all it's functionality? [yes/no] (Be honest) yes absolutely
  • Does your code fit the style guidelines? [yes/no] yes
  • Does your PR fit the contribution guidelines? [yes/no] yes

@GhzGarage GhzGarage self-assigned this Nov 11, 2023
@GhzGarage GhzGarage added the enhancement New feature or request label Nov 11, 2023
@gypsychicken
Copy link

One suggestion I have, would be to set a spawn point for each location, so it can be use for larger operations vehicles or boats or even airplanes/helis

    label = 'Police Garage', -- Garage label
    locations= { -- Take out / Spawn locations you can add as many as you want
        [1] = { take = vector3(443.41, -1021.34, 28.55), spawn = vector4(438.59, -1025.28, 28.78, 4.47) },
    },

Copy link

This PR has had 60 days of inactivity & will close within 7 days

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

Successfully merging this pull request may close these issues.

3 participants