search

brentN5 / bt-target

GNU General Public License v3.0
36 stars 52 forks source link

Added job checking with support for multiple jobs. #5

Closed LiamDormon closed 3 years ago

LiamDormon commented 3 years ago

By default the config is set to use ESX, but if disables allows the user to return their framework's PlayerJob object.

deenz-dev commented 3 years ago

Its not working mate, take a look at it again :/

LiamDormon commented 3 years ago

Do you get an error? This is working for me on Non-ESX. Edit: Fixed syntax error left over from when I copied it to github.

deenz-dev commented 3 years ago

Screenshot_1 Maybe cause the line 12? an "if" outside a function... xd

LiamDormon commented 3 years ago

Should be fixed by loading the config before the client script in the fxmanifest, tried and tested this morning without issues

deenz-dev commented 3 years ago

I had some errors anyways, can i add you on Discord?

LiamDormon commented 3 years ago

AFAIK everything in this build is working, tried and tested.

deenz-dev commented 3 years ago

Screenshot_2

deenz-dev commented 3 years ago

Also, update README, job = {"police", "ambulance", "mechanic"}, <-- not working without this (comma, obv) distance = 1.5

LiamDormon commented 3 years ago

You must be doing something wrong because this works :) I loaded it up on my ESX server and my QB Server and on both it returned correctly.

deenz-dev commented 3 years ago

i didnt change anything tbh