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 documentation regarding TaskPlaneMission #1215

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

Conversation

colistro123
Copy link
Contributor

@colistro123 colistro123 commented Oct 22, 2024

Updates the native documentation regarding TaskPlaneMission to reflect the eVehicleMission enum as well as the updated native parameters.

This also updates certain native parameter types that were wrong (not float, but int as depicted in the game's idb dating back to 323, not sure why that was wrong, I guess due to assumptions).

Updates the native documentation regarding TaskPlaneMission to reflect
the `eVehicleMission` enum as well as the updated native parameters.

This also updates certain native parameter types that were wrong, certain parameters are
not `float`, but `int`.
```

### Added parameters
* **bPrecise**: A boolean value, defaults to `true`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This can be added as a parameter

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done, thanks!

TASK/TaskPlaneMission.md Outdated Show resolved Hide resolved
colistro123 and others added 2 commits November 5, 2024 06:28
Move bPrecise to parameter section.
@AvarianKnight
Copy link
Collaborator

Thanks!

I will triple check tomorrow that this properly builds tomorrow before I merge, as native-doc-toolings might still be out of date in the main repo.

@AvarianKnight AvarianKnight added the merging-blocked PR is waiting for something else to be merged. label Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merging-blocked PR is waiting for something else to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants