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 New LS Tuners DLC Vehicles #258

Merged
merged 3 commits into from
Aug 15, 2021
Merged

Add New LS Tuners DLC Vehicles #258

merged 3 commits into from
Aug 15, 2021

Conversation

MichaelCoding25
Copy link
Contributor

  • Add new Tuners DLC vehicles
  • Update DLC descriptions to be more accurate and include minimum build required to use them
  • Remove notes about DLCs which are default build of fivem or before (Arena War)

- Add new Tuners DLC vehicles
- Update DLC descriptions to be more accurate and include minimum build required to use them
- Remove notes about DLCs which are default build of fivem or before (Arena War)
@Blumlaut
Copy link

Blumlaut commented Jul 26, 2021

There are some cars missing:

rt3000  (Sports)
warrener2 (Sedans)

@MichaelCoding25
Copy link
Contributor Author

You're right @Blumlaut I was using the RageMP wiki, so it did not have the above ones, also the DOMINATOR8 was missing

@ReadyP1ayerTwo
Copy link

How about the track rims included in the update? They can be found in OpenIV in the directory as shown below; hopefully this helps. :) Thank you michael.

GTA V > update > x64 > dlcpacks > mptuner > dlc.rpf > x64 > levels > mptuner > vehiclemods > smod5wheels_mods.rpf

@MichaelCoding25
Copy link
Contributor Author

@ReadyP1ayerTwo As far as I know, because they are a part of the tuner category they automatically should be available on the new cars.

@ReadyP1ayerTwo
Copy link

ReadyP1ayerTwo commented Jul 28, 2021

@ReadyP1ayerTwo As far as I know, because they are a part of the tuner category they automatically should be available on the new cars.

No sir; It's a completely new category called track! They also added a new category called street in the last update which is currently in vMenu

Edited for preview: https://youtu.be/yyV2ckrHViQ?t=19

@MichaelCoding25
Copy link
Contributor Author

@ReadyP1ayerTwo As far as I know, because they are a part of the tuner category they automatically should be available on the new cars.

No sir; It's a completely new category called track! They also added a new category called street in the last update which is currently in vMenu

Edited for preview: https://youtu.be/yyV2ckrHViQ?t=19

Gotcha, do you know by any chance what the ID of the new category is? Because it's not documented in https://docs.fivem.net/natives/?_0xB3ED1BFB4BE636DC

@Blumlaut
Copy link

@ReadyP1ayerTwo As far as I know, because they are a part of the tuner category they automatically should be available on the new cars.

No sir; It's a completely new category called track! They also added a new category called street in the last update which is currently in vMenu
Edited for preview: https://youtu.be/yyV2ckrHViQ?t=19

Gotcha, do you know by any chance what the ID of the new category is? Because it's not documented in https://docs.fivem.net/natives/?_0xB3ED1BFB4BE636DC

i'd assume its 12, assuming they just incrementally count upwards

@ReadyP1ayerTwo
Copy link

@ReadyP1ayerTwo As far as I know, because they are a part of the tuner category they automatically should be available on the new cars.

No sir; It's a completely new category called track! They also added a new category called street in the last update which is currently in vMenu
Edited for preview: https://youtu.be/yyV2ckrHViQ?t=19

Gotcha, do you know by any chance what the ID of the new category is? Because it's not documented in https://docs.fivem.net/natives/?_0xB3ED1BFB4BE636DC

I'd have to go with what Blum said; It only makes sense :)

@MichaelCoding25
Copy link
Contributor Author

I mean yeah lol, I already tested and pushed it if you saw.

Copy link

@Blumlaut Blumlaut left a comment

Choose a reason for hiding this comment

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

Works perfectly.

@AJDevelopment1
Copy link

When will this be merged?

@MichaelCoding25
Copy link
Contributor Author

When will this be merged?

Whenever @TomGrobbe decided it's time to do that

@TomGrobbe TomGrobbe changed the base branch from master to development August 15, 2021 13:59
@TomGrobbe TomGrobbe merged commit 92c5386 into TomGrobbe:development Aug 15, 2021
@MichaelCoding25 MichaelCoding25 deleted the add_lstuner_vehicles branch November 5, 2021 16:14
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.

None yet

5 participants