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

Park in new garage / dry boat spawn #48

Merged
merged 5 commits into from Sep 13, 2019

Conversation

CryptoGenics
Copy link
Contributor

Solves issue with parking a vehicle in a new garage and still having to pay transfer fee.

Solves issue with parking a vehicle in a new garage and still having to pay transfer fee.
@TanguyOrtegat
Copy link
Owner

thanks for your improvments! bu where is server file ? and please could you bundle your changes ? thanks

@CryptoGenics
Copy link
Contributor Author

CryptoGenics commented Aug 29, 2019

There are no changes for the server files. Just took the trigger "esx_eden_garage:MoveGarage" already used when paying to change garage location, and added it to the vehicle and trailer return.

@CryptoGenics
Copy link
Contributor Author

Added Boat Spawn marker so you do not have to swim to spawn in your boat.

@TarisGN
Copy link

TarisGN commented Aug 31, 2019

i've got this error when i put a car in anither garage than the one that i tranfered :
[esx_jb_eden_garage2] An error happens on MySQL for query "UPDATE owned_vehicles SET garage_name =@garage_name WHERE plate='HES 728'": ER_BAD_NULL_ERROR: Column 'garage_name' cannot be null�[0m

@CryptoGenics
Copy link
Contributor Author

With my fix? Did you change the garage to garage_name in the client 247 and 264 as well as add the move garage trigger?

@CryptoGenics
Copy link
Contributor Author

Might be an issue with your garage_name in SQL

@TarisGN
Copy link

TarisGN commented Sep 1, 2019

Yup, i took your whole client file.

@CryptoGenics
Copy link
Contributor Author

Fixed

@CryptoGenics CryptoGenics changed the title Park in new garage Park in new garage / dry boat spawn Sep 5, 2019
@TanguyOrtegat TanguyOrtegat merged commit ea32663 into TanguyOrtegat:master Sep 13, 2019
@TanguyOrtegat
Copy link
Owner

thanks for contributing to the script and make it better :)

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

Successfully merging this pull request may close these issues.

None yet

3 participants