Skip to content

Conversation

@Tyler-9822
Copy link
Contributor

@Tyler-9822 Tyler-9822 commented Sep 2, 2025

This is to add the configurable functionality to prevent a player from adding or removing extras on a vehicle in order to repair it.

Config options consist of:

  • Enabling the core functionality
  • The amount of engine damage that can be tolerated before the extras are blocked
  • The amount of body damage that can be tolerated before the extras are blocked
    - Enabling a notification to the player when they attempt to change extras, but are not permitted
    - The contents of the notification

@Tyler-9822 Tyler-9822 changed the title Prevent Abuse of Vehicle Extras Menu to Repair Vehicles Prevent Abuse of Vehicle Extras Menu to Repair Vehicles and Fix Weather Syncing Sep 6, 2025
Copy link
Collaborator

@cm8263 cm8263 left a comment

Choose a reason for hiding this comment

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

Thanks for your PR @Tylerg9822-sudo! I've left a few comments regarding changes. Let me know if I can help any further. Cheers!

@Tyler-9822 Tyler-9822 marked this pull request as draft September 20, 2025 15:35
@Tyler-9822 Tyler-9822 changed the title Prevent Abuse of Vehicle Extras Menu to Repair Vehicles and Fix Weather Syncing Prevent Abuse of Vehicle Extras Menu to Repair Vehicles Sep 20, 2025
@Tyler-9822 Tyler-9822 requested a review from cm8263 September 20, 2025 17:58
remove option to disable notification per request
remove option to disable notification per request
remove option to disable notification per request
@Tyler-9822 Tyler-9822 marked this pull request as ready for review September 25, 2025 23:26
@cm8263
Copy link
Collaborator

cm8263 commented Sep 26, 2025

Thanks again @Tylerg9822-sudo. I added an extra check that will disable the items in the menu if the vehicle is too damaged prior to opening the menu - it looks like so:

image

If a player opens the menu and then changes an extra, your check kicks in and prevents the extra change 👍

@cm8263 cm8263 changed the base branch from master to development September 26, 2025 05:15
@cm8263 cm8263 merged commit 822228f into TomGrobbe:development Sep 26, 2025
2 checks passed
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.

3 participants