Skip to content
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

document speech params and names #494

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

coalaura
Copy link

Copy link
Contributor

@colistro123 colistro123 left a comment

Choose a reason for hiding this comment

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

LGTM at a glance.

content/docs/game-references/speeches.md Outdated Show resolved Hide resolved
Comment on lines 11 to 14
``` lua
-- Makes the ped shout an insult
PlayPedAmbientSpeechNative(ped, "GENERIC_INSULT_HIGH"--[[speechName]], "SPEECH_PARAMS_FORCE"--[[speechParam]])
```
Copy link
Contributor

Choose a reason for hiding this comment

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

Never forget people will copy / paste and expect the "example" to work. You shoud define ped.

Copy link
Author

Choose a reason for hiding this comment

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

The speech natives (from my experience) don't seem to really work on player peds, so there would need to be a decent bit of "boilerplate" code to find a working ped. Unless you have a different idea?

Copy link
Contributor

Choose a reason for hiding this comment

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

Does it not work directly on a random created ped?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Just use https://docs.fivem.net/natives/?_0xB4AC7D0CF06BFE8F to create a random ped at the players current position

content/docs/game-references/speeches.md Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants