Skip to content

Allow client peds to enter/exit client vehicles #3678

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

Closed
wants to merge 14 commits into from

Conversation

gownosatana
Copy link
Contributor

This pull request allows client created peds to enter client created vehicles.

setPedEnterVehicle(Element ped, [Element vehicle, bool passenger])
When no vehicle is specified ped will enter closest client vehicle

Test resource: test.zip

@tederis tederis added the enhancement New feature or request label Aug 28, 2024
@FileEX
Copy link
Contributor

FileEX commented Sep 2, 2024

Shouldn't we also use the onClientPedVehicleExit / onClientPedVehicleEnter events for peds here?

@gownosatana
Copy link
Contributor Author

gownosatana commented Sep 2, 2024

Shouldn't we also use the onClientPedVehicleExit / onClientPedVehicleEnter events for peds here?

yep, it works here too, it's tested

@gownosatana
Copy link
Contributor Author

All done

@gownosatana
Copy link
Contributor Author

gownosatana commented Sep 6, 2024

@tederis review please ✨

@FileEX
Copy link
Contributor

FileEX commented Sep 6, 2024

Why do you ask for a review if you don't acknowledge it? You're not following the coding guidelines and you're marking conversations as resolved even though you haven't done anything 🤷

@gownosatana
Copy link
Contributor Author

Why do you ask for a review if you don't acknowledge it? You're not following the coding guidelines and you're marking conversations as resolved even though you haven't done anything 🤷

I acknowledge you may have reading problems but I made everything you asked for in conversations, just didn't answer to it and pushed commit

@gownosatana
Copy link
Contributor Author

Also I just want to add you're able to make comments and reviews but you're not one deciding here if pull request should be merged or not, thanks 🙏

@TracerDS
Copy link
Contributor

TracerDS commented Sep 6, 2024

Why do you ask for a review if you don't acknowledge it? You're not following the coding guidelines and you're marking conversations as resolved even though you haven't done anything 🤷

I acknowledge you may have reading problems but I made everything you asked for in conversations, just didn't answer to it and pushed commit

Actually, you didnt. You pointlessly resolved conversations without adding the requested changes

@gownosatana
Copy link
Contributor Author

gownosatana commented Sep 6, 2024

Why do you ask for a review if you don't acknowledge it? You're not following the coding guidelines and you're marking conversations as resolved even though you haven't done anything 🤷

I acknowledge you may have reading problems but I made everything you asked for in conversations, just didn't answer to it and pushed commit

Actually, you didnt. You pointlessly resolved conversations without adding the requested changes

I did in commit e461e70, I'm sorry for your blindness, other changes are approved by fileex.

Copy link
Contributor

@FileEX FileEX left a comment

Choose a reason for hiding this comment

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

This PR already looks good, which unfortunately cannot be said about the others

@TracerDS
Copy link
Contributor

TracerDS commented Sep 6, 2024

I did in commit e461e70, I'm sorry for your blindness, other changes are approved by fileex.

Being rude to others does not benefit you whatsoever. You only added the type but did not remove the braces; i.e. You acknowledged 1 out of 3 conversations.

@Dutchman101
Copy link
Member

Dutchman101 commented Sep 6, 2024

@gownosatana Generally speaking, over all your recent PR's and code reviews on it, you are creating a lot of unnecesary drama, and you don't want to agree to do things the way this project has implemented.

You are this close to an organisation (repository) ban and the deletion of all your PR's, so im just warning you to keep yourself in check. Do not reply to this comment. Start showing some more respect, and cut the drama/combativeness.

You're not gonna get things the way you want it; the world or this project doesn't revolve around you. If you don't like that, you are free to just go away.

@CrosRoad95

This comment was marked as abuse.

@Dutchman101
Copy link
Member

This PR is being closed for the same reason as the comment explains on the other one closed.

@Dutchman101 Dutchman101 closed this Oct 5, 2024
@multitheftauto multitheftauto locked as too heated and limited conversation to collaborators Oct 5, 2024
@botder botder modified the milestone: 1.6.1 Dec 22, 2024
@botder botder added this to the 1.6.1 milestone Jan 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants