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

Procedural Avionics Unlock costs #682

Merged
merged 5 commits into from Jun 10, 2017
Merged

Procedural Avionics Unlock costs #682

merged 5 commits into from Jun 10, 2017

Conversation

rsparkyc
Copy link
Member

@rsparkyc rsparkyc commented Jun 8, 2017

This addresses the following issues:
#679
#677
#676
#630

Since upgrades now require you to unlock them, it's best to start a new vessel in the VAB, add a procedural avionics unit, right click on Show GUI, and purchase the available upgrades. (You just need to buy the last one in each category. Buying a higher level automatically purchases the ones below it - their cost is calculated into the cost of the higher level)

Crafts already in flight should be grandfathered in, but to build new vessels, you'll need to unlock the new tech.

@pjf pjf added the in progress label Jun 8, 2017
@rsparkyc
Copy link
Member Author

rsparkyc commented Jun 8, 2017

@pap1723 @leudaimon (@ec429?) I think you guys were using procedural avionics, so you might want to look at this

{
ProceduralAvionicsUtils.Log("Unlocking ", techNodeName, " for ", avionicsConfigName);
if (!unlockedTech.ContainsKey(avionicsConfigName)) {
ProceduralAvionicsUtils.Log("Unlocking for the fist time");
Copy link
Contributor

Choose a reason for hiding this comment

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

'fist' is presumably meant to be 'first'?

Copy link
Member

Choose a reason for hiding this comment

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

Fist

@rsparkyc rsparkyc merged commit 70a72dd into KSP-RO:master Jun 10, 2017
@pjf pjf removed the in progress label Jun 10, 2017
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

4 participants