-
Notifications
You must be signed in to change notification settings - Fork 375
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
Duty Blip change when inside a vehicle #431
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks Good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the reason for the local variable? could use if IsPedInAnyVehicle(ped) then
as its not used again?
There can be more additions done using this local variable and I don't think using a local variable to define that could be a big issue bruh
|
No big issue at all. just pointless code bud |
Yeah Actually Local Variable help you avoid cluttering But If It's Only one Then It's Also Fine He Was Just Suggesting You right? @Jamie9192 |
And If A Config would Be There It would be Pretty Nice 👍🏻 |
moved duty-blips to QB-Jobs |
@r0adra93 Check qb-jobs PR added it there too for all the jobs but for time being until its unavailable to use people can use this for now and until then it can be merged too unless the code is wiped from here |
This PR has had 60 days of inactivity & will close within 7 days |
This PR is for enhancing QOL stuff and main objective for the change is
To differentiate the duty blip from in a vehicle and out of vehicle
So when you sit inside a vehicle the blip turns into a car and heads accordingly and vice versa when on foot it stays default.