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

AB compat rail/scope height update #5650

Merged
merged 2 commits into from
Oct 23, 2017
Merged

Conversation

dedmen
Copy link
Contributor

@dedmen dedmen commented Oct 21, 2017

Compat variant of #5648
I only updated the weapons we already had in compat and only added new ones were really necessary.
I noticed we are missing all RHS AK variants with NPZ which have a different railheight that the non-NPZ ones.
If someone else want's to add the missing stuff here is a table of all rail heights: https://docs.google.com/spreadsheets/d/1d5O_nC0Cq26CWGI4sqjZU9jdhArV5pkSm-VMDT7uvM4/edit?usp=sharing

@ulteq ulteq added area/config kind/enhancement Release Notes: **IMPROVED:** labels Oct 21, 2017
@dedmen
Copy link
Contributor Author

dedmen commented Oct 21, 2017

@ulteq What is WIP on this? Besides adding all the NPZ classes which I don't plan on doing.

@ulteq
Copy link
Contributor

ulteq commented Oct 22, 2017

Besides adding all the NPZ classes

This, but I guess we could also insert them later with a separate PR.

@dedmen
Copy link
Contributor Author

dedmen commented Oct 22, 2017

Yeah. This is just a update to improve the existing compat. IMO the RHS compat should be completly overhauled. checking if inheritance is still correct. and really adding all the missed classes

@jonpas jonpas added this to the 3.11.0 milestone Oct 22, 2017
@ulteq ulteq removed the status/WIP label Oct 22, 2017
@jonpas jonpas merged commit a614744 into acemod:master Oct 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Release Notes: **IMPROVED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants