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

Update TaskFollowPointRoute.md #943

Merged
merged 8 commits into from
Aug 7, 2024
Merged

Conversation

Wetter42
Copy link
Contributor

Hopefully this follows the recent advice given, nonetheless, roastme! :D

Added flags and updated the name of the intunknown

Before you submit this PR, please make sure:

  • You have read the contribution guidelines
  • You include an example that validates your change
  • Your English is grammatically correct

Added flags and updated the name of the intunknown
@Wetter42
Copy link
Contributor Author

RIP - I guess thats a no... 😢

@ahcenezdh
Copy link
Contributor

The last argument should be mode and you have to declare the enum by wrap it into a c block c // Enum here
You could move the description of the native at the top

Makes the ped go on the created point route

@ahcenezdh
Copy link
Contributor

CI failed because you didn't write followpointroutemode in the same way as in the parameters. Anyway, the correct name of this parameter is mode.

Made requested changes...hope this follows the pattern needed
@Wetter42
Copy link
Contributor Author

Done - Added changes. please approve for moneys

@@ -5,8 +5,15 @@ ns: TASK

```c
// 0x595583281858626E 0xB837C816
void TASK_FOLLOW_POINT_ROUTE(Ped ped, float speed, int unknown);
void TASK_FOLLOW_POINT_ROUTE(Ped ped, float speed, int mode);
Copy link
Collaborator

Choose a reason for hiding this comment

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

routeMode would be a better name

TASK/TaskFollowPointRoute.md Outdated Show resolved Hide resolved
(Hopefully) fixed enum so that it's formatted properly!
Used the above example...as an example
Committing to change to the last parameter name pending any additional enum changes needed
Hopefully this is the appropriate change.  Still willing to try again if not. Thanks!
TASK/TaskFollowPointRoute.md Outdated Show resolved Hide resolved
Clean up old remnants and port over usage example.
TASK/TaskFollowPointRoute.md Outdated Show resolved Hide resolved
TASK/TaskFollowPointRoute.md Outdated Show resolved Hide resolved
@AvarianKnight AvarianKnight merged commit d483169 into citizenfx:master Aug 7, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants