-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
NetworkCreateSynchronisedScene improvements #922
Conversation
netScene id | ||
Network scene handle. This can be used to start / stop it and add peds or entities to it. | ||
|
||
## Examples: |
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.
Examples cannot have a colon (:
).
## Advanced example | ||
You can find an advanced example [here](https://pastebin.com/masLVdm5) |
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.
This should be on the description, above parameters section. Also, 'double hash' (##
) custom headers like these are not allowed, but 'triple hash' headers are. (###
), so ideally ### Advanced example
.
It's also somewhat discouraged to use pastebin links since they can expire/get removed, causing the research to be lost.
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.
Thank you for your contribution, however, some changes are needed. Please follow the guidelines outlined in CONTRIBUTING.md.
Thank you 🙂
local pedCo = GetEntityCoords(ped) | ||
local sceneObject = GetClosestObjectOfType(pedCo, 2.0, GetEntityModel('prop_weed_01'), false, false, false) |
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.
Small nitpick, I think changing pedCo
to pedCoords
would be better for readability here.
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.
I'm going to change everything you just told me
Improvements: