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

switchable MKS containers #125

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

Grimmas
Copy link
Contributor

@Grimmas Grimmas commented Apr 25, 2021

Adding four switchable container wedges - regular and small containers for MKS solids and liquids.

They reuse the models from the ore and rocket fuel containers.

This relies on the other patch for FUR that I just submitted - Nils277/FelineUtilityRovers#58

Edit: I said MKS but strictly speaking they should work and be of use with base CRP as well, even if MKS is missing - thanks to the clever way you set up the stuff with the resource switch templates in KerbetrotterTools.

@Nils277
Copy link
Owner

Nils277 commented May 2, 2021

I will keep this one open until KPBS has undergone the company takeover to Kerbetrotter.
Can you please add to the parts that they need MKS to be available? They might confuse users that have not installed MKS and wonder that they need these resources for.

@Grimmas
Copy link
Contributor Author

Grimmas commented May 3, 2021

I think that's already taken care of in the Kerbetrotter templates. IE if someone installs CRP and Kerbetrotter he would get the two liquids wedges but without MKS he would only get to store LF/Ox/Mono. And in the case of the solids wedges he would only get to store Ore. The additional resources are enabled not in the part but in your Kerbetrotter templates and encompass not only MKS but also Pathfinder, NFT, Ose, EL, etc

So rather than MKS containers these are more like CRP++ containers. That's why I made them depend on both CRP and Kerbetrotter DLL instead of on MKS.

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.

2 participants