Skip to content

Conversation

@adhershmnair
Copy link
Contributor

@adhershmnair adhershmnair commented Nov 16, 2024

Vehicle Options Menu - Added "Vehicle Moddings"
image

Vehicle Moddings Menu
image

Modifications Menu
image

Vehicle Color Menu
image

Fixed
image

Pending Task

Add Lang in other language files.

RegisterServerEvent('qb-admin:giveWeapon', function(weapon)
local src = source
if QBCore.Functions.HasPermission(src, 'admin') or IsPlayerAceAllowed(src, 'command') then
local Player = QBCore.Functions.GetPlayer(src)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removing as it is an unused variable.

@adhershmnair adhershmnair changed the title Removed unused variable to fix lint issues Added Vehicle Modification options. Nov 17, 2024
@@ -0,0 +1,1055 @@
VehicleMods = {};
VehicleMods.enabled = true;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Disable the Vehicle Moddings Menu.

@@ -0,0 +1,1055 @@
VehicleMods = {};
VehicleMods.enabled = true;
VehicleMods.FuelResource = "LegacyFuel";
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Switch the FuelResource if something else is used.

VehicleMods = {};
VehicleMods.enabled = true;
VehicleMods.FuelResource = "LegacyFuel";
VehicleMods.colors = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Vehicle color list with index.

colorindex = 112
}
};
VehicleMods.metalcolors = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Mettalic color list.

colorindex = 120
}
};
VehicleMods.mattecolors = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Matte Color list

colorindex = 154
}
};
VehicleMods.tyrecolors = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tyre Color List

b = 128
}
};
VehicleMods.neoncolors = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Neon Light color List

description = "Demo Item 4"
}
};
VehicleMods.vehmods = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Vehicle modification configuration list.

@Qwerty1Verified
Copy link
Member

Hello, I've been looking at this PR over the last few days and gathering feedback. We're currently deciding what to do in regards to the resource so it might take a little longer for me to get back to you.

Thank you for your work on the PR and I hope to get back to you soon.

@ckaysone380
Copy link

love the addons for the veh mod to the admin menu, testing it now currently, so far so good with everything working as normal

@JericoFX
Copy link

i think this admin menu needs to use qb menu instead of menuv

@JericoFX
Copy link

image
image

@QBStevie
Copy link

QBStevie commented Jan 16, 2025

i think this admin menu needs to use qb menu instead of menuv

i do agree Mate maybe get a pr in for it

Copy link

@Bellck Bellck left a comment

Choose a reason for hiding this comment

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

Love this, would be better in QB-Menu rather than MenuV for an upgrade.

@adhershmnair
Copy link
Contributor Author

Thanks for the suggestions! I agree that moving towards qb-menu would align better with the QBCore ecosystem. However, this current implementation uses checkboxes and sliders - UI components not currently supported by qb-menu.

Once qb-menu adds support for those elements, we can definitely work on upgrading and refactoring the admin menu to use it instead. Until then, MenuV allows us to maintain the required functionality.

Happy to revisit this once qb-menu catches up feature-wise! 👍

FYI @Qwerty1Verified @GhzGarage

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.

6 participants