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

add docs to SET_VEHICLE_LIGHTS #1187

Closed
wants to merge 1 commit into from

Conversation

coalaura
Copy link
Contributor

@coalaura coalaura commented Sep 8, 2024

Resolved the enum for SET_VEHICLE_LIGHTS and added a note for potential name to 0x1FD09E7390A74D54

@AvarianKnight AvarianKnight self-assigned this Sep 10, 2024
@@ -10,6 +10,8 @@ void _SET_VEHICLE_LIGHTS_MODE(Vehicle vehicle, int p1);
```

```
Could be SET_VEHICLE_HEADLIGHT_SHADOWS?
Copy link
Collaborator

Choose a reason for hiding this comment

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

Any reason not to just rename this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not entirely sure about the name, was just a guess since its location kinda lined up, functionality wise i didn't test it

Copy link
Collaborator

Choose a reason for hiding this comment

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

Would be better off just not changing the file then imo.

@AvarianKnight AvarianKnight added the needs validation This looks good, but needs additional confirmation of suggested change. label Sep 11, 2024
@coalaura
Copy link
Contributor Author

Closing in favor of #1192

@coalaura coalaura closed this Sep 21, 2024
@coalaura coalaura deleted the vehicle-lights branch September 21, 2024 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs validation This looks good, but needs additional confirmation of suggested change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants