-
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
docs(natives/PED): Change documentation SET_PED_CONFIG_FLAGS #917
Conversation
Deleting the entire enum seems a bit weird, you should just update the current enums on the ones that changed. Also please read the contributor guidelines, you shouldn't link to external websites as the links get deleted |
There are too many of them and they don't fit properly. To read them, you have to scroll horizontally and vertically or zoom out the browser. On the other hand, many links of the pastebin website are in the documentation of some natives. For example, this one: However, if you say, I will put the whole list in the form of an array as before. |
A lot of the ones in the list are already documented and are known true (any with CPED_CONFIG_FLAG_[name] are hash matches iirc, you'' notice some that have comments beside them because they likely don't match)
The doc you listed are from when FiveM did their original Natives import, any that link to outside websites should be replaced with in-line docs or link to the citizenfx/fivem-docs |
I updated all the values in the same array as before. |
Idk how accurate this source is but there seem to be a few more named ones here |
Replaced with a better source
The source you mentioned was better and had more flags that I added. Thanks |
_0x262A3B8E = 462, | ||
_0x1AA79A25 = 463 |
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.
Not sure about that. I can't seem to found aything about that.
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'll PR the rest missing flag when this if get merged. I check them all and seem's the be all valid.
This PR seems to have the full list #1082 |
Superseded by #1082 |
Adding a complete list of flags with string names